Skip to content

[3.0][cdc-common] Change DataChangeEvent from interface to class, and add implementation for DataChangeEvent.#2677

Merged
GOODBOY008 merged 3 commits intoapache:masterfrom
joyCurry30:2665_add_implementation_datachange_event
Nov 9, 2023
Merged

[3.0][cdc-common] Change DataChangeEvent from interface to class, and add implementation for DataChangeEvent.#2677
GOODBOY008 merged 3 commits intoapache:masterfrom
joyCurry30:2665_add_implementation_datachange_event

Conversation

@joyCurry30
Copy link
Copy Markdown
Contributor

Refer to #2665
Change DataChangeEvent from interface to class, and add implementation for DataChangeEvent.
PTAL. @lvyanquan @GOODBOY008

@joyCurry30 joyCurry30 force-pushed the 2665_add_implementation_datachange_event branch from d06a4ab to e449112 Compare November 9, 2023 13:57
@joyCurry30 joyCurry30 force-pushed the 2665_add_implementation_datachange_event branch from e449112 to f943ec8 Compare November 9, 2023 13:59
Copy link
Copy Markdown
Member

@GOODBOY008 GOODBOY008 left a comment

Choose a reason for hiding this comment

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

Thanks for you contritbutions, I left a minor comment.

Copy link
Copy Markdown
Contributor

@lvyanquan lvyanquan left a comment

Choose a reason for hiding this comment

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

+1

@GOODBOY008 GOODBOY008 merged commit de45676 into apache:master Nov 9, 2023
@joyCurry30 joyCurry30 deleted the 2665_add_implementation_datachange_event branch November 10, 2023 02:01
ChaomingZhangCN pushed a commit to ChaomingZhangCN/flink-cdc that referenced this pull request Jan 13, 2025
… add implementation for DataChangeEvent. (apache#2677)

* [3.0][cdc-common] Change DataChangeEvent from interface to class, and add implementation for DataChangeEvent.

* [3.0][cdc-common] Remove @param for code cleaning.

* [3.0][cdc-common] Make the DataChangeEvent constructor private.
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.

3 participants