Skip to content

Conversation

plaes
Copy link

@plaes plaes commented Aug 25, 2025

Changes

  • Add optional --ignored-tables [filename] argument to specify file containing list of tables to be ignored.

Checklist

  • You've added tests (in tests/) which would fail without your patch
  • You've added a new changelog entry (in CHANGES.rst).

@plaes plaes marked this pull request as draft August 25, 2025 21:24
@agronholm
Copy link
Owner

What is the use case where you have so many tables to ignore that it'd be impractical to list them on the command line or to just list the tables you do want?

@plaes
Copy link
Author

plaes commented Aug 26, 2025

What is the use case where you have so many tables to ignore that it'd be impractical to list them on the command line or to just list the tables you do want?

Well, there are 910 tables in the ignore list (I want to ignore tables that are empty), and after that I am left with 622 tables which have some data...

My usecase - investigating an old ERP system (development seems to be started some time after 1991 where foreign keys did not yet exist). ;)

@sheinbergon
Copy link
Collaborator

@agronholm I don't mind accepting such a feature

@agronholm
Copy link
Owner

At the very least the options need to be consistent. If --tables accepts a comma-separated list of tables but --ignore-tables (which would intuitively feel like its negative counterpart) accepts a path instead, they're not consistent.

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.

3 participants