Skip to content

feat(ingest/snowflake): cross account lineage through snowflake shares#16748

Draft
alokr-dhub wants to merge 3 commits intomasterfrom
feature/snowflake-cross-account-lineage
Draft

feat(ingest/snowflake): cross account lineage through snowflake shares#16748
alokr-dhub wants to merge 3 commits intomasterfrom
feature/snowflake-cross-account-lineage

Conversation

@alokr-dhub
Copy link
Contributor

  • The PR conforms to DataHub's Contributing Guideline (particularly PR Title Format)
  • Links to related issues (if applicable)
  • Tests for the changes have been added/updated (if applicable)
  • Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
  • For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@github-actions github-actions bot added the ingestion PR or Issue related to the ingestion of metadata label Mar 24, 2026
@codecov
Copy link

codecov bot commented Mar 24, 2026

Codecov Report

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

Files with missing lines Patch % Lines
...ingestion/source/snowflake/snowflake_schema_gen.py 75.00% 5 Missing ⚠️
...hub/ingestion/source/snowflake/snowflake_shares.py 93.75% 5 Missing ⚠️

📢 Thoughts on this report? Let us know!

@alwaysmeticulous
Copy link

alwaysmeticulous bot commented Mar 24, 2026

✅ Meticulous spotted 0 visual differences across 1526 screens tested: view results.

Meticulous evaluated ~8 hours of user flows against your PR.

Expected differences? Click here. Last updated for commit 1f7ce22. This comment will update as new commits are pushed.

@codecov
Copy link

codecov bot commented Mar 24, 2026

Bundle Report

Changes will decrease total bundle size by 131 bytes (-0.0%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
datahub-react-web-esm 22.68MB -131 bytes (-0.0%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: datahub-react-web-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-*.js -131 bytes 12.45MB -0.0%

@datahub-connector-tests
Copy link

Connector Tests Results

Connector tests failed for commit c5f7d20

View full test logs →

To skip connector tests, add the skip-connector-tests label (org members only).

Autogenerated by the connector-tests CI pipeline.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant