-
Notifications
You must be signed in to change notification settings - Fork 0
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
Allow ignoring of certain upstream lineage platforms #15
Conversation
@@ -438,6 +444,11 @@ def __init__( | |||
# when emitting custom SQL data sources. | |||
self.custom_sql_ids_being_used: List[str] = [] | |||
|
|||
if self.config.ignore_upstream_lineage_platforms: | |||
self.ignore_upstream_lineage_platforms = ( | |||
self.config.ignore_upstream_lineage_platforms.split(",") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we also trim in case people use spaces?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point!
9df12c2
to
f51f59a
Compare
304ceca
to
d6cc196
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
* Allow ignoring of certain upstream lineage platforms * Handle whitespace * Remove lineage_platform_instance
* Allow ignoring of certain upstream lineage platforms * Handle whitespace * Remove lineage_platform_instance
* Allow ignoring of certain upstream lineage platforms * Handle whitespace * Remove lineage_platform_instance
* Allow ignoring of certain upstream lineage platforms * Handle whitespace * Remove lineage_platform_instance
* Allow ignoring of certain upstream lineage platforms * Handle whitespace * Remove lineage_platform_instance
Adds the ability to ignore certain platforms for upstream lineage.