Skip to content

[FLINK-39289][paimon] Supports executing TRUNCATE schema change events on tables with deletion-vectors.enabled:true set#4327

Merged
lvyanquan merged 3 commits intoapache:masterfrom
gongzexin:FLINK-39289
Mar 24, 2026
Merged

[FLINK-39289][paimon] Supports executing TRUNCATE schema change events on tables with deletion-vectors.enabled:true set#4327
lvyanquan merged 3 commits intoapache:masterfrom
gongzexin:FLINK-39289

Conversation

@gongzexin
Copy link
Contributor

Currently, TRUNCATE schema change events are not allowed on Paimon tables where deletion-vectors.enabled:true is set, while Paimon does not have this restriction, and this should be allowed.

…s on tables with deletion-vectors.enabled:true set.
@gongzexin
Copy link
Contributor Author

Regarding the issue we discussed in the DingTalk community group today, would @lvyanquan like to take a look?

@sd4324530
Copy link
Contributor

@gongzexin Overall it's good. Please also modify the corresponding unit tests.

…he error check for the paimon table where deletion-vectors.enabled:true is set.
@gongzexin
Copy link
Contributor Author

@gongzexin Overall it's good. Please also modify the corresponding unit tests.

As you said, I modified the corresponding unit test. would @sd4324530 like to take a look?

@lvyanquan
Copy link
Contributor

Run 'mvn spotless:apply' to fix these violations.

@gongzexin
Copy link
Contributor Author

Run 'mvn spotless:apply' to fix these violations.

The violation has been resolved. Do @lvyanquan have any other suggestions?

Copy link
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.

@lvyanquan lvyanquan merged commit 08f3fb8 into apache:master Mar 24, 2026
19 checks passed
Mrart pushed a commit to Mrart/flink-cdc that referenced this pull request Mar 26, 2026
…s on tables with deletion-vectors.enabled:true set (apache#4327)

Co-authored-by: hector.gong <hector.gong@feisu.com>
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.

3 participants