Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for excluding tables #182

Merged
merged 10 commits into from
Jul 13, 2022
Merged

Support for excluding tables #182

merged 10 commits into from
Jul 13, 2022

Conversation

alexbruy
Copy link
Contributor

@alexbruy alexbruy commented Jul 5, 2022

Add command-line option --skip-tables TABLES to exclude some tables from opeations, tables passed as semicolon delimeted string, e.g --skip-tables table1;table2;...tableN.

If set, specified tables will be ignore when:

  • creating changeset;
  • applying changeset;
  • rebasing;
  • dumping contents;
  • retrieving schema;
  • copying tables between different drivers

This is work in progress and any feedback is welcome.

geodiff/src/geodiff-cli.cpp Outdated Show resolved Hide resolved
geodiff/src/geodiff-cli.cpp Outdated Show resolved Hide resolved
geodiff/src/geodiff-cli.cpp Show resolved Hide resolved
geodiff/src/geodiffcontext.cpp Outdated Show resolved Hide resolved
geodiff/src/geodiff.h Outdated Show resolved Hide resolved
geodiff/src/drivers/sqlitedriver.cpp Outdated Show resolved Hide resolved
geodiff/src/drivers/postgresdriver.cpp Outdated Show resolved Hide resolved
pygeodiff/main.py Show resolved Hide resolved
pygeodiff/tests/test_skip_tables.py Show resolved Hide resolved
@alexbruy alexbruy force-pushed the skip-tables branch 2 times, most recently from 6abe550 to 26fba10 Compare July 12, 2022 10:53
Copy link
Contributor

@wonder-sk wonder-sk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the updates, just few minor bits

geodiff/src/drivers/postgresdriver.cpp Outdated Show resolved Hide resolved
geodiff/src/drivers/postgresdriver.cpp Outdated Show resolved Hide resolved
geodiff/src/drivers/sqlitedriver.cpp Outdated Show resolved Hide resolved
geodiff/src/geodiff.cpp Outdated Show resolved Hide resolved
geodiff/src/geodiff.h Outdated Show resolved Hide resolved
Copy link
Contributor

@wonder-sk wonder-sk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done 🎉

@wonder-sk wonder-sk merged commit 38ec23d into master Jul 13, 2022
@alexbruy alexbruy deleted the skip-tables branch July 15, 2022 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants