Skip to content

feat(sql_parsing): add generate_column_lineage parameter to skip CLL computation#16244

Open
kyungsoo-datahub wants to merge 1 commit intomasterfrom
dev/sigma-skip-column-lineage
Open

feat(sql_parsing): add generate_column_lineage parameter to skip CLL computation#16244
kyungsoo-datahub wants to merge 1 commit intomasterfrom
dev/sigma-skip-column-lineage

Conversation

@kyungsoo-datahub
Copy link
Contributor

Add generate_column_lineage parameter (default True) to sqlglot_lineage API, allowing callers to skip expensive column-level lineage computation when only table-level lineage is needed.

Sigma source now passes generate_column_lineage=False since it only uses table-level lineage (ChartInfoClass.inputs), avoiding unnecessary CLL computation.

Also use named arguments in sqlglot_lineage() calls for clarity.

…computation

Add generate_column_lineage parameter (default True) to sqlglot_lineage
API, allowing callers to skip expensive column-level lineage computation
when only table-level lineage is needed.

Sigma source now passes generate_column_lineage=False since it only uses
table-level lineage (ChartInfoClass.inputs), avoiding unnecessary CLL
computation.

Also use named arguments in sqlglot_lineage() calls for clarity.
@github-actions github-actions bot added the ingestion PR or Issue related to the ingestion of metadata label Feb 17, 2026
@github-actions
Copy link
Contributor

Linear: ING-1651

@codecov
Copy link

codecov bot commented Feb 17, 2026

Codecov Report

❌ Patch coverage is 80.00000% with 3 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...gestion/src/datahub/sql_parsing/sqlglot_lineage.py 85.71% 2 Missing ⚠️
...estion/src/datahub/ingestion/source/sigma/sigma.py 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ingestion PR or Issue related to the ingestion of metadata needs-review Label for PRs that need review from a maintainer.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments