-
Notifications
You must be signed in to change notification settings - Fork 3k
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(ingest): unity-catalog - Unity catalog source #6292
Conversation
…ata/datahub-fork into master+acr-4552-mvp-unity-catalog
- Stateful ingestion - Filtering - More verbose reporting - Domain support - Connection test support - Complex type support
Moving config into separate file
Unit Test Results (metadata ingestion) 8 files ± 0 8 suites ±0 54m 34s ⏱️ - 2m 17s For more details on these failures and errors, see this check. Results for commit 96bd74c. ± Comparison against base commit 46453e4. ♻️ This comment has been updated with latest results. |
catalog: str | ||
|
||
|
||
class UnitySchemaKey(CatalogKey): |
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.
I'm not a big fan of integration specific classes being added to mcp_builder.py ... we have accumulated a few and need to remove these later IMO
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.
LGTM!
Co-authored-by: MohdSiddique Bagwan <[email protected]> Co-authored-by: Shirshanka Das <[email protected]>
Co-authored-by: MohdSiddique Bagwan <[email protected]> Co-authored-by: Shirshanka Das <[email protected]>
Checklist