-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[Feature] exclude option for check_cols snapshot #10438
Comments
hey, if this feature is still relevant i'd be keen to have a go at it this weekend. not contributed here yet but have a bit of time and should be able to get something together |
That would be great @will-sylvera! Definitely still relevant :) |
working on this now -- doing it from my personal GH account not my work one |
@graciegoheen hey sorry, slightly dumb q here: whats your process for raising PRs across multiple repos? I have to make changes here and in the adapters repo here. should I just raise PRs in both places at once? |
Yep! You can just raise both PRs at once, and we will review them together |
This issue has been marked as Stale because it has been open for 180 days with no activity. If you would like the issue to remain open, please comment on the issue or else it will be closed in 7 days. |
Although we are closing this issue as stale, it's not gone forever. Issues can be reopened if there is renewed community interest. Just add a comment to notify the maintainers. |
Is this your first time submitting a feature request?
Describe the feature
Right now snapshots have
check_cols
- we could add a new config for the inverse, something likecheck_exclude_cols
.This would be similar to the incremental model config -
merge_exclude_columns
Describe alternatives you've considered
No response
Who will this benefit?
No response
Are you interested in contributing this feature?
No response
Anything else?
No response
The text was updated successfully, but these errors were encountered: