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

ENH Exclude a list of tables for checking and rapairs #10496

Closed

Conversation

ssmarco
Copy link
Contributor

@ssmarco ssmarco commented Sep 8, 2022

Issue #10495

@GuySartorelli
Copy link
Member

GuySartorelli commented Sep 12, 2022

should this new configuration be based on defining explicit table names?
Why not use DataObject class names and infer the table names from that?
Why not define an upper limit of records, and any table with more than that number of records gets skipped?

@ssmarco
Copy link
Contributor Author

ssmarco commented Sep 13, 2022

should this new configuration be based on defining explicit table names? Why not use DataObject class names and infer the table names from that? Why not define an upper limit of records, and any table with more than that number of records gets skipped?

Click the wrong button

@ssmarco ssmarco closed this Sep 13, 2022
@ssmarco
Copy link
Contributor Author

ssmarco commented Sep 13, 2022

should this new configuration be based on defining explicit table names?
Why not use DataObject class names and infer the table names from that?

Great idea will update the PR.

Why not define an upper limit of records, and any table with more than that number of records gets skipped?

My goal is to lessen the amount of queries to be executed.

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.

2 participants