Skip to content

Conversation

@hawk9821
Copy link
Contributor

@hawk9821 hawk9821 commented Sep 3, 2024

Purpose of this pull request

Paimon Sink supported truncate table

Does this PR introduce any user-facing change?

How was this patch tested?

org.apache.seatunnel.e2e.connector.paimon.PaimonSinkCDCIT#testSinkPaimonTruncateTable
org.apache.seatunnel.e2e.connector.paimon.PaimonSinkHdfsIT#testSinkPaimonHdfsTruncateTable
1725353064340
org.apache.seatunnel.e2e.connector.paimon.PaimonSinkHdfsIT#testSinkPaimonHiveTruncateTable
17253528098823

Check list

@Hisoka-X
Copy link
Member

Hisoka-X commented Sep 3, 2024

cc @dailai

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.

Could you rename all test case file. I can not get the difference between the config file through name.

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.

Thinks @hawk9821 LGTM

@hawk9821
Copy link
Contributor Author

hawk9821 commented Sep 4, 2024

Could you rename all test case file. I can not get the difference between the config file through name.

get

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.

@Hisoka-X Hisoka-X changed the title [Feature][CONNECTORS-V2-Paimon] Paimon Sink supported truncate table [Feature][Connector-V2] Paimon Sink supported truncate table Sep 4, 2024
@hailin0 hailin0 merged commit 4f3df22 into apache:dev Sep 4, 2024
@hawk9821 hawk9821 deleted the paimon_truncate branch September 19, 2024 00:46
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