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

[Fix][Connector-V2] Fix paimon e2e error #7721

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

hawk9821
Copy link
Contributor

Purpose of this pull request

fix paimon e2e bug
c267899aeb9c28eade23e699875a564

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

Copy link
Member

@Hisoka-X Hisoka-X left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @hawk9821 !

@Hisoka-X Hisoka-X changed the title [HotFix][CONNECTORS-V2-Paimon] paimon e2e error [Fix][Connector-V2] Fix paimon e2e error Sep 23, 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.
This can not trigger the dynamic bucket logical by default. Otherwise, there should be some problem when write a same table with multiple times in dynamic bucket logical. The author will submit another pr to fix the problem. Link pr #7335

@TyrantLucifer TyrantLucifer merged commit 61d1964 into apache:dev Sep 23, 2024
8 checks passed
PeppaPage pushed a commit to PeppaPage/seatunnel that referenced this pull request Sep 23, 2024
PeppaPage pushed a commit to PeppaPage/seatunnel that referenced this pull request Sep 23, 2024
@hawk9821 hawk9821 deleted the hotfix_paimon_bucketAssigner branch September 25, 2024 07:50
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