Skip to content

Conversation

@Stefanietry
Copy link
Contributor

@Stefanietry Stefanietry commented Jan 14, 2026

[core]

Purpose

Linked issue: #7037

When re-overwrite data to delta branch, delete the corresponding partition on snapshot branch.

Tests

org.apache.paimon.spark.SparkChainTableITCase#testChainTable
org.apache.paimon.spark.SparkChainTableITCase#testHourlyChainTable

@Stefanietry Stefanietry force-pushed the support_batch_recall_for_chain_table branch 2 times, most recently from e7a7fe0 to c7cd982 Compare January 15, 2026 08:55
(dynamicPartitionOverwriteMode, overwritePartition)
}

private def postCommit(
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you should add CommitCallback instead of modifying Spark.

Copy link
Contributor Author

@Stefanietry Stefanietry Jan 19, 2026

Choose a reason for hiding this comment

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

Get, thanks for your suggestion. I have rewritten this function using the CommitCallback mechanism.

@Stefanietry Stefanietry force-pushed the support_batch_recall_for_chain_table branch from c7cd982 to d7a945e Compare January 19, 2026 02:51
@Stefanietry Stefanietry force-pushed the support_batch_recall_for_chain_table branch from d7a945e to 626fefa Compare January 19, 2026 07:24
Copy link
Contributor

@JingsongLi JingsongLi left a comment

Choose a reason for hiding this comment

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

+1

@JingsongLi JingsongLi merged commit 84a552b into apache:master Jan 19, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants