Skip to content

Conversation

@dlg99
Copy link
Contributor

@dlg99 dlg99 commented Sep 27, 2021

Motivation

Add documentation for new connector.

Modifications

Added documentation.

Verifying this change

  • Make sure that the change passes the CI checks.

This change is a documentation only.

Does this pull request potentially affect one of the following parts:

If yes was chosen, please highlight the changes

No

Documentation

Check the box below and label this PR (if you have committer privilege).

Need to update docs?

  • doc

Co-authored-by: Michael Marshall <mikemarsh17@gmail.com>
Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

Great!

@Anonymitaet PTAL

Debezium [requires](https://debezium.io/documentation/reference/1.5/connectors/oracle.html#oracle-overview) Oracle DB with LogMiner or XStream API enabled.
Supported options and steps for enabling them vary from version to version of Oracle DB.
Steps outlined in the [documentation](https://debezium.io/documentation/reference/1.5/connectors/oracle.html#oracle-overview) and used in the [integration test](https://github.com/apache/pulsar/blob/master/tests/integration/src/test/java/org/apache/pulsar/tests/integration/io/sources/debezium/DebeziumOracleDbSourceTester.java) may or may not work for the version and edition of Oracle DB you are using.
Please refer to the documentation for Oracle DB as needed.
Copy link
Member

Choose a reason for hiding this comment

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

Can you add a reference link here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Steps outlined in the [documentation](https://debezium.io/documentation/reference/1.5/connectors/oracle.html#oracle-overview) and used in the [integration test](https://github.com/apache/pulsar/blob/master/tests/integration/src/test/java/org/apache/pulsar/tests/integration/io/sources/debezium/DebeziumOracleDbSourceTester.java) may or may not work for the version and edition of Oracle DB you are using.
Please refer to the documentation for Oracle DB as needed.

Similarly to other connectors, you can use JSON or YAMl to configure the connector.
Copy link
Member

Choose a reason for hiding this comment

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

Like other connector docs, can you give a JSON example?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@Anonymitaet Anonymitaet added the doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. label Sep 28, 2021
dlg99 and others added 2 commits September 28, 2021 09:20
CR feedback

Co-authored-by: Anonymitaet <50226895+Anonymitaet@users.noreply.github.com>
@Anonymitaet Anonymitaet added this to the 2.9.0 milestone Sep 29, 2021
@Anonymitaet Anonymitaet merged commit 73bf69e into apache:master Sep 29, 2021
@dlg99 dlg99 deleted the oracle_doc branch October 14, 2021 23:31
@dlg99 dlg99 mentioned this pull request Nov 12, 2021
1 task
bharanic-dev pushed a commit to bharanic-dev/pulsar that referenced this pull request Mar 18, 2022
* Documentation for Oracle Debezium connector

* Update site2/docs/io-debezium-source.md

Co-authored-by: Michael Marshall <mikemarsh17@gmail.com>

* Apply suggestions from code review

CR feedback

Co-authored-by: Anonymitaet <50226895+Anonymitaet@users.noreply.github.com>

* CR feedback

Co-authored-by: Michael Marshall <mikemarsh17@gmail.com>
Co-authored-by: Anonymitaet <50226895+Anonymitaet@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. release/2.9.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants