Skip to content

Comments

[INLONG-5315][Sort] Import all changelog mode data ingest into JDBC#5316

Merged
dockerzhang merged 3 commits intoapache:masterfrom
thesumery:5315
Aug 8, 2022
Merged

[INLONG-5315][Sort] Import all changelog mode data ingest into JDBC#5316
dockerzhang merged 3 commits intoapache:masterfrom
thesumery:5315

Conversation

@thesumery
Copy link
Contributor

@thesumery thesumery commented Aug 1, 2022

Prepare a Pull Request

Import all changelog mode data ingest into JDBC

Motivation

JDBC does not support CDC sink without primary key, so we could give an option to regard all changelog data as insert-only data. So we can insert into jdbc from canal-json,mysql-cdc etc.

Modifications

add a sink.ignore.changelog option to support insert-only mode

Copy link
Contributor

@yunqingmoswu yunqingmoswu left a comment

Choose a reason for hiding this comment

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

LGTM

@dockerzhang dockerzhang merged commit 6ee93e7 into apache:master Aug 8, 2022
bruceneenhl pushed a commit to bruceneenhl/inlong that referenced this pull request Aug 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature][Sort] Import all changelog mode data ingest into JDBC

5 participants