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

DBZ-2049 Add deployment steps to the doc for each connector #1801

Merged
merged 11 commits into from Sep 11, 2020

Conversation

TovaCohen
Copy link
Contributor

These updates are for DBZ-2049.
I used the deployment instructions that we updated in the PostgreSQL connector doc as the model to update and add deployment instructions for each connector.
There is a separate commit for the updates in each connector.
I manually copied the updated files downstream to make sure that the downstream book builds. You might notice a few updates that seem unrelated to this update, but I had to make them to get the book to build.
In the PostgreSQL connector doc, I updated a link target URL.
I confirmed that Antora builds the doc.
These updates are not required for 1.2, but it would be nice to be able to provide them in the downstream 1.2 doc.
Let me know what needs to change.

Copy link
Member

@gunnarmorling gunnarmorling left a comment

Choose a reason for hiding this comment

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

Nice work! A few comments inline; most of them should apply to each occurrence of that part in the different connectors' docs.

documentation/modules/ROOT/pages/connectors/mongodb.adoc Outdated Show resolved Hide resolved
documentation/modules/ROOT/pages/connectors/mongodb.adoc Outdated Show resolved Hide resolved
documentation/modules/ROOT/pages/connectors/mongodb.adoc Outdated Show resolved Hide resolved
documentation/modules/ROOT/pages/connectors/mongodb.adoc Outdated Show resolved Hide resolved
documentation/modules/ROOT/pages/connectors/mongodb.adoc Outdated Show resolved Hide resolved
documentation/modules/ROOT/pages/connectors/sqlserver.adoc Outdated Show resolved Hide resolved
@TovaCohen
Copy link
Contributor Author

Note that for the Db2, MySQL, and SQL Server connector doc, I also changed the format that explains the connector configuration settings. For those connectors only, the rendered downstream user guide had duplicated callout number in the explanatory text below the code samples. I tried many things to fix this, with no luck. So I removed the problem text and replaced it with a table.

@Naros Naros merged commit eec5719 into debezium:master Sep 11, 2020
@Naros
Copy link
Member

Naros commented Sep 11, 2020

LGTM, merged to master and branch 1.2

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