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

Rdsc 2368 add to the signaling table doc a section for oracle #3256

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

galilev
Copy link
Contributor

@galilev galilev commented Jul 30, 2024

No description provided.

@galilev galilev requested a review from spicy-sauce July 30, 2024 13:22
@spicy-sauce spicy-sauce added the rdi Redis Data Integration label Jul 30, 2024
@@ -114,14 +114,74 @@ The data-collections array lists tables by their fully-qualified names, using th

#### Signaling Table Columns
Copy link
Member

Choose a reason for hiding this comment

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

In general, I think Redis' convention for headers is like this:
Signalling table columns.

@andy-stark-redis please, confirm.
In this case, we should apply to all headers and sub-headers in this doc.

Comment on lines +136 to +138
id VARCHAR(42) PRIMARY KEY,
type VARCHAR(32) NOT NULL,
data VARCHAR(2048) NULL
Copy link
Member

Choose a reason for hiding this comment

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

Indent one tab to the left.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

k,done

);
```

2. Add the property `debezium.source.signal.data.collection` to the `application.properties` file:
Copy link
Member

Choose a reason for hiding this comment

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

This is outdated, you should explain how to add this to the advanced section, no more application.properties.


> Note: If the supplemental logging is enabled for the entire database you can skip this step.

4. Restart the Debezium Server.
Copy link
Member

Choose a reason for hiding this comment

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

Show how.


## Example for Adding the `CUSTOMERS` table to the pipeline

1. Add the `CUSTOMERS` table to the `debezium.source.table.include.list` property in the `application.properties` file:
Copy link
Member

Choose a reason for hiding this comment

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

application.properties is obsolete, use the new format of config.yaml.

debezium.source.table.include.list=C##DBZUSER.PRODUCTS,C##DBZUSER.ORDERS,C##DBZUSER.CUSTOMERS
```

2. Enable supplemental logging for the `CUSTOMERS`` table:
Copy link
Member

Choose a reason for hiding this comment

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

redundant closing `.


2. Enable supplemental logging for the `CUSTOMERS`` table:

To enable supplemental logging for all the table columns:
Copy link
Member

Choose a reason for hiding this comment

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

remove, you have the same in line 168.

ALTER TABLE C##DBZUSER.CUSTOMERS ADD SUPPLEMENTAL LOG DATA(ALL) COLUMNS
```

3. Restart the `Debezium Server`.
Copy link
Member

Choose a reason for hiding this comment

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

show how.

VALUES ('1', 'execute-snapshot', '{"data-collections":["ORCLPDB1.C##DBZUSER.CUSTOMERS"],"type":"incremental"}');
```

> Note: The column `id` is a unique string in the DEBEZIUM_SIGNAL table
Copy link
Member

Choose a reason for hiding this comment

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

DEBEZIUM_SIGNAL --> DEBEZIUM_SIGNAL.

@spicy-sauce
Copy link
Member

Wrong repository, open a new PR here:
https://github.com/redis/docs

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

Successfully merging this pull request may close these issues.

2 participants