Skip to content

Conversation

xuzifu666
Copy link
Member

minor code refractor in BinaryRecordDataSerializer

@xuzifu666 xuzifu666 changed the title [Rumtime] hotfix in BinaryRecordDataSerializer code refractor [Rumtime] Hotfix in BinaryRecordDataSerializer with code refractor Dec 10, 2023
Copy link
Contributor

@joyCurry30 joyCurry30 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 the patch! LGTM.

@xuzifu666
Copy link
Member Author

Thanks for the patch! LGTM.

Thanks for the patch! LGTM.

cc @joyCurry30 CI error seems not related to the changed code

@xuzifu666 xuzifu666 requested a review from joyCurry30 December 10, 2023 08:09
Copy link
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 @xuzifu666 for the contribution, LGTM

@leonardBang leonardBang merged commit e6bb8e9 into apache:master Dec 13, 2023
@leonardBang leonardBang added this to the V3.1.0 milestone Dec 13, 2023
lvyanquan pushed a commit that referenced this pull request Jan 18, 2024
…aSerializer (#2845)

Co-authored-by: xuyu <[email protected]>
(cherry picked from commit e6bb8e9)
joyCurry30 pushed a commit to joyCurry30/flink-cdc-connectors that referenced this pull request Mar 22, 2024
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants