-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
feat(ingestion/redshift): support auto_incremental_lineage #9010
feat(ingestion/redshift): support auto_incremental_lineage #9010
Conversation
…ub.com:siddiquebagwan-gslab/datahub-fork into master+ing-188-sqlglotlineage-for-cll-redshift
metadata-ingestion/src/datahub/ingestion/source/redshift/redshift.py
Outdated
Show resolved
Hide resolved
metadata-ingestion/setup.py
Outdated
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.
need an addition to changelog + tweak to make incremental_lineage off by default for redshift
it should also be clear that to use both CLL and incremental lineage, you need to provide a graph instance / use the rest sink
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.
clicked the wrong button
…ift.py Co-authored-by: Harshal Sheth <hsheth2@gmail.com>
…siddiquebagwan-gslab/datahub-fork into master+incremental-lineage-for-redshift
@siddiquebagwan-gslab looks like there's merge conflicts on this - it's my bad for not merging it earlier |
…siddiquebagwan-gslab/datahub-fork into master+incremental-lineage-for-redshift
No description provided.