Skip to content

[cdc-composer] fix CI and initial Context with existed config from SourceDef and SinkDef.#2749

Merged
leonardBang merged 2 commits intoapache:masterfrom
lvyanquan0624:ci_composer
Nov 24, 2023
Merged

[cdc-composer] fix CI and initial Context with existed config from SourceDef and SinkDef.#2749
leonardBang merged 2 commits intoapache:masterfrom
lvyanquan0624:ci_composer

Conversation

@lvyanquan
Copy link
Copy Markdown
Contributor

@lvyanquan lvyanquan commented Nov 24, 2023

Configuration provide a convenient way for DataSourceFactory to get ConfigOption.

@PatrickRen @leonardBang CC.

@lvyanquan lvyanquan changed the title [cdc-composer] avoid NoPointException while getting option from Context. [cdc-composer] fix CI and initial Context with existed config from SourceDef and SinkDef. Nov 24, 2023
Copy link
Copy Markdown
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

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

Thanks @lvyanquan for the contribution, a test is welcome

@lvyanquan
Copy link
Copy Markdown
Contributor Author

a test is welcome

Address it.

@leonardBang leonardBang added this to the V3.0.0 milestone Nov 24, 2023
Copy link
Copy Markdown
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

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

Thanks @lvyanquan for the update, LGTM, wait the ci green

@leonardBang leonardBang merged commit a26607a into apache:master Nov 24, 2023
e-mhui pushed a commit to e-mhui/flink-cdc-connectors that referenced this pull request Dec 2, 2023
ChaomingZhangCN pushed a commit to ChaomingZhangCN/flink-cdc that referenced this pull request Jan 13, 2025
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.

2 participants