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

[Feature][CONNECTORS-V2-Paimon] Piamon Sink supports changelog-procuder is lookup and full-compaction mode #7834

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

hawk9821
Copy link
Contributor

@hawk9821 hawk9821 commented Oct 14, 2024

Purpose of this pull request

Piamon Sink supports writing to the Piamon table whose changelog-procuder is lookup and full-compaction mode.

Does this PR introduce any user-facing change?

How was this patch tested?

e2e: PaimonSinkCDCIT#testChangelogLookup PaimonSinkCDCIT#testChangelogFullCompaction

Check list

@Hisoka-X
Copy link
Member

cc @dailai

@hawk9821 hawk9821 force-pushed the paimon_change_producer branch 2 times, most recently from 9bf3fd5 to dd69259 Compare October 16, 2024 08:17
@hawk9821 hawk9821 changed the title [Feature][CONNECTORS-V2-Paimon] changelog-producer support the lookup and full-compaction mode [Feature][CONNECTORS-V2-Paimon] Piamon Sink supports changelog-procuder is lookup and full-compaction mode Oct 16, 2024
@hawk9821 hawk9821 force-pushed the paimon_change_producer branch 2 times, most recently from 6bd8322 to 91b055d Compare October 17, 2024 13:26
dailai
dailai previously approved these changes Oct 18, 2024
Copy link
Contributor

@dailai dailai left a comment

Choose a reason for hiding this comment

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

LGTM

@dailai
Copy link
Contributor

dailai commented Oct 18, 2024

Please fix ci.

dailai
dailai previously approved these changes Oct 18, 2024
hailin0
hailin0 previously approved these changes Oct 18, 2024
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.

4 participants