Skip to content
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

MGDCTRS-1869 MGDCTRS-1870 Add missing config options to connector catalog #29

Merged
merged 2 commits into from
Jan 18, 2023

Conversation

ani-sha
Copy link
Contributor

@ani-sha ani-sha commented Jan 17, 2023

@ani-sha ani-sha requested a review from Naros January 17, 2023 14:30
@ani-sha ani-sha marked this pull request as draft January 17, 2023 14:34
@Naros
Copy link
Contributor

Naros commented Jan 17, 2023

Hi @ani-sha, I added a comment to MGDCTRS-1870. Ideally, we need an upstream change to the schema generation tooling to specify which debezium-storage-xxxx module's configuration needs to be loaded and appended to each of the connector's configuration definitions to return to what we had with Debezium 1.x.

@ani-sha ani-sha marked this pull request as ready for review January 18, 2023 11:03
Copy link
Contributor

@Naros Naros left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @ani-sha, LGTM.

@ani-sha ani-sha merged commit f46065f into bf2fc6cc711aee1a0c2a:main Jan 18, 2023
@@ -459,7 +473,7 @@ objects:
"icon_href" : "http://example.com/images/debezium-postgres-2.0.1.Final.png",
"schema" : {
"title" : "Debezium PostgreSQL Connector",
"required" : [ "topic.prefix", "database.hostname", "database.user", "database.dbname" ],
"required" : [ "topic.prefix", "database.hostname", "database.user" ],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Naros is database.names really an optional config option?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants