Skip to content
This repository has been archived by the owner on May 17, 2024. It is now read-only.

closing connection once data diff is executed #785

Merged
merged 52 commits into from
Jan 22, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
52 commits
Select commit Hold shift + click to select a range
55be7fe
closing connection once data diff is executed
sar009 Nov 28, 2023
78827e8
typo fix
sar009 Nov 29, 2023
d309b98
moving database to with block
sar009 Nov 29, 2023
fe49f40
optimizing _data_diff function
sar009 Nov 29, 2023
933a393
minor
sar009 Nov 29, 2023
dc0147a
minor
sar009 Nov 29, 2023
a916183
bug fix
sar009 Nov 29, 2023
f1a8baa
linter fixes
sar009 Nov 29, 2023
e263027
formating code
sar009 Nov 29, 2023
3dbb6da
update CONTRIBUTING.md and cleanup ansi escape sequences
sar009 Dec 1, 2023
8871d16
Merge branch 'datafold:master' into 784
sar009 Dec 1, 2023
82d2674
Merge branch 'master' into 784
sar009 Dec 6, 2023
ba4182e
Merge branch 'master' into 784
sar009 Dec 21, 2023
8cf6e88
Merge branch 'master' into 784
sar009 Dec 24, 2023
193c2bd
Merge branch 'master' into 784
sar009 Dec 29, 2023
89b2ab7
Merge branch 'master' into 784
sar009 Dec 29, 2023
d78e8e0
Merge branch '784' of github.com:sar009/data-diff into 784
sar009 Dec 30, 2023
4699c66
Merge branch 'master' of github.com:sar009/data-diff into 784
sar009 Dec 30, 2023
3de6c3a
read postgres db config from common.CONN_STRINGS
sar009 Dec 31, 2023
6828c4a
Merge branch 'master' into 784
sar009 Jan 1, 2024
9ac5967
Merge branch 'master' into 784
sungchun12 Jan 2, 2024
1d6c371
defaulting postgres port
sar009 Jan 3, 2024
00aa1be
Merge branch '784' of github.com:sar009/data-diff into 784
sar009 Jan 3, 2024
73c63e0
Merge branch 'master' into 784
sar009 Jan 3, 2024
e9273c4
Merge branch 'master' into 784
sungchun12 Jan 3, 2024
59a9e8b
Merge branch 'master' into 784
sungchun12 Jan 4, 2024
08edfa4
Merge branch 'master' into 784
sar009 Jan 5, 2024
e9a7630
Merge branch 'master' into 784
sar009 Jan 8, 2024
7280344
Merge branch 'master' into 784
sar009 Jan 9, 2024
13bbd5c
Merge branch 'master' into 784
sar009 Jan 10, 2024
3ed4bce
test
sar009 Jan 10, 2024
b2a1542
test for __main__
sar009 Jan 10, 2024
6a86647
adding close db connection test
sar009 Jan 10, 2024
0b9d360
update test case
sar009 Jan 10, 2024
9bf91ce
do not use shared connection
sar009 Jan 10, 2024
9f4ac6e
no connection sharing
sar009 Jan 10, 2024
78d0976
no connection sharing
sar009 Jan 10, 2024
9bd278f
avoid list typing
sar009 Jan 10, 2024
44bddad
Merge branch 'master' into 784
sar009 Jan 11, 2024
4397d7e
Merge branch 'master' into 784
sar009 Jan 12, 2024
a611ccc
Merge branch 'master' into 784
sungchun12 Jan 12, 2024
10c6f36
Update tests/test_main.py
sar009 Jan 14, 2024
2333dfc
Update tests/test_main.py
sar009 Jan 14, 2024
3ec84e0
Apply suggestions from code review
sar009 Jan 14, 2024
ab19d2a
Update tests/test_database.py
sar009 Jan 14, 2024
6d00f42
minor
sar009 Jan 14, 2024
4513be9
minor
sar009 Jan 15, 2024
4c4203d
minor
sar009 Jan 15, 2024
10f6afa
minor
sar009 Jan 15, 2024
09d33a9
Merge branch 'master' into 784
sungchun12 Jan 15, 2024
75713f9
merging 784
sar009 Jan 16, 2024
dba5623
remove redundant variable
sar009 Jan 18, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -149,4 +149,6 @@ benchmark_*.png
.vscode

# History
.history
.history

docker-compose-local.yml
11 changes: 10 additions & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ Make sure to update the appropriate `TEST_*_CONN_STRING`, so that it will be inc

#### Run the tests

You can run the tests with `unittest`.
You can run the tests with `python -m unittest`.

When running against multiple databases, the tests can take a long while.

Expand Down Expand Up @@ -111,6 +111,15 @@ $ poetry install # Install dependencies
$ docker-compose up -d mysql postgres # run mysql and postgres dbs in background
```

If you want to change the configuration of docker-compose and run the DB containers, copy docker-compose.yml into docker-compose-local.yml, make changes and run
```shell-session
$ cp docker-compose.yml docker-compose-local.yml
$ docker-compose -f docker-compose-local.yml up -d mysql postgres # run mysql and postgres dbs in background
```
you will also have to set up `tests/local_settings.py` where `TEST_*_CONN_STRING` can be edited

`docker-compose-local.yml` and `tests/local_settings.py` is git ignored so should not show up in git changes.

[docker-compose]: https://docs.docker.com/compose/install/

**3. Run Unit Tests**
Expand Down
Loading