apache
[Feature]Implement replace branch in BranchManager
Purpose
Linked issue: close https://github.com/apache/incubator-paimon/issues/2153
Tests
BranchActionITCase.testReplaceBranchToTargetBranch
API and Format
Documentation
@sunxiaojian Can you split the merge and replace branch in different PRs? I think the operations between them are quite different
Can you split the merge and replace branch in different PRs? I think the operations between them are quite different
@FangYongs But they may use some of the same methods added in SnapshotManager and TagManager. The main methods for BranchManager are mergeBranch and replaceBranch.
https://github.com/apache/incubator-paimon/pull/2862 Looks like there's been some conflict, do I need to close or change this pr?
#2862 Looks like there's been some conflict, do I need to close or change this pr?
@TaoZex No need to close it. I will delete the merge branch section of this PR, and it will be based on yours.
#2862 Looks like there's been some conflict, do I need to close or change this pr?
@TaoZex No need to close it. I will delete the merge branch section of this PR, and it will be based on yours.
Thanks for your reply.
@sunxiaojian Can you split the merge and replace branch in different PRs? I think the operations between them are quite different
@FangYongs The merge branch has been removed, and the implementation of the merge branch is mainly based on #2862
@FangYongs Please help to review this pr also. And I suggest the merge/replace branches could wait for the Flink read/write pr merged and do detailed tests base on Flink reading/writing, because the merging and replacing branches will impact on the main branch, which is risky. WDYT?
@FangYongs @schnappi17 Is there anything else that needs to be modified?