-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
fixes #12771: Added Greenplum Ingestion Connector #13128
Conversation
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
@chirag-madlani @karanh37 Could you please also review and give the approve for this pull request? We are really looking forward to the support of Greenplum in OpenMetadata 🙏 |
@evmuravev @T01iK Can you resolve the conflicts here? |
resolved conflicts and retested the connector locally |
[open-metadata-ui] Kudos, SonarCloud Quality Gate passed! |
[open-metadata-ingestion] SonarCloud Quality Gate failed. 0 Bugs 0.0% Coverage Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
Describe your changes:
fixes #12771 : Added Greenplum Ingestion Connector (based on Postgres connector from OpenMetadata 1.1 but queries updated to support Greenplum).
Tested on Greenplum v6
Added Greenplum connector in Ingestion, Test Connection Service, UI, JsonSchema
Connector was needed to support Greenplum in my project
Using Metadata CLI and Airflow on production Greenplum in my company
Type of change:
Checklist:
Fixes <issue-number>: <short explanation>