Skip to content

Conversation

@acrylJonny
Copy link
Collaborator

  • Dashboard-level lineage (table-to-dashboard via datasetEdges)
  • Native SQL query lineage parsing
  • Query builder lineage extraction
  • Nested query lineage (card-to-card references)
  • Collection tags on dashboards, charts, and models
  • Metabase Models as Dataset entities
  • Comprehensive unit and integration tests

…dels

- Dashboard-level lineage (table-to-dashboard via datasetEdges)
- Native SQL query lineage parsing
- Query builder lineage extraction
- Nested query lineage (card-to-card references)
- Collection tags on dashboards, charts, and models
- Metabase Models as Dataset entities
- Comprehensive unit and integration tests
@codecov
Copy link

codecov bot commented Nov 10, 2025

Codecov Report

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

Files with missing lines Patch % Lines
...ingestion/src/datahub/ingestion/source/metabase.py 84.66% 23 Missing ⚠️

📢 Thoughts on this report? Let us know!

@alwaysmeticulous
Copy link

alwaysmeticulous bot commented Nov 10, 2025

✅ Meticulous spotted visual differences in 2 of 939 screens tested, but all differences have already been approved: view differences detected.

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

Last updated for commit b14800f. This comment will update as new commits are pushed.

@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label Nov 10, 2025
@codecov
Copy link

codecov bot commented Nov 10, 2025

Bundle Report

Bundle size has no change ✅

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.

2 participants