Skip to content

feat(sdk): add get_lineage #13654

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.

Sign up for GitHub

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

Jump to bottom
Merged
yoonhyejin merged 30 commits into from
Jun 6, 2025
Merged

feat(sdk): add get_lineage #13654

yoonhyejin merged 30 commits into from
Jun 6, 2025

Conversation

@yoonhyejin
Copy link
Collaborator

@yoonhyejin yoonhyejin commented May 29, 2025 •
edited
Loading

Add get_lineage method that reads column_level / table_level lineage

@yoonhyejin yoonhyejin requested a review from hsheth2 May 29, 2025 13:44
@github-actions github-actions bot added the ingestion PR or Issue related to the ingestion of metadata label May 29, 2025
@codecov
Copy link

codecov bot commented May 29, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 91.54930% with 6 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...tadata-ingestion/src/datahub/sdk/lineage_client.py 91.54% 6 Missing ⚠️

📢 Thoughts on this report? Let us know!

@yoonhyejin yoonhyejin marked this pull request as ready for review May 29, 2025 13:53
@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label May 29, 2025
yoonhyejin
yoonhyejin commented Jun 2, 2025
View reviewed changes
hsheth2
hsheth2 reviewed Jun 2, 2025
View reviewed changes
@datahub-cyborg datahub-cyborg bot added pending-submitter-response Issue/request has been reviewed but requires a response from the submitter and removed needs-review Label for PRs that need review from a maintainer. labels Jun 2, 2025
Base automatically changed from sdk/write-lineage to master June 5, 2025 06:26
@datahub-cyborg datahub-cyborg bot added needs-review Label for PRs that need review from a maintainer. and removed pending-submitter-response Issue/request has been reviewed but requires a response from the submitter labels Jun 5, 2025
hsheth2
hsheth2 approved these changes Jun 5, 2025
View reviewed changes
@datahub-cyborg datahub-cyborg bot added pending-submitter-merge and removed needs-review Label for PRs that need review from a maintainer. labels Jun 5, 2025
@yoonhyejin yoonhyejin merged commit e169b4a into master Jun 6, 2025
56 of 57 checks passed
@yoonhyejin yoonhyejin deleted the sdk/read-lineage branch June 6, 2025 03:34
kartikey-visa pushed a commit to kartikey-visa/datahub that referenced this pull request Jul 23, 2025
@yoonhyejin @kartikey-visa
feat(sdk): add get_lineage (datahub-project#13654)
6cab029
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@hsheth2 hsheth2 hsheth2 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

ingestion PR or Issue related to the ingestion of metadata pending-submitter-merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yoonhyejin @hsheth2