Skip to content

Commit

Permalink
[BugFix] Fix snapshot versions in cross-cluster replication (#54790)
Browse files Browse the repository at this point in the history
Signed-off-by: xiangguangyxg <[email protected]>
(cherry picked from commit c6d3689)
  • Loading branch information
xiangguangyxg authored and mergify[bot] committed Jan 8, 2025
1 parent f6337a5 commit c320b4c
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion be/src/storage/replication_txn_manager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,9 @@ Status ReplicationTxnManager::remote_snapshot(const TRemoteSnapshotRequest& requ
}

std::vector<Version> missed_versions;
tablet->calc_missed_versions(request.src_visible_version, &missed_versions);
for (auto v = request.visible_version + 1; v <= request.src_visible_version; ++v) {
missed_versions.emplace_back(v, v);
}
if (UNLIKELY(missed_versions.empty())) {
LOG(WARNING) << "Remote snapshot tablet skipped, no missing version"
<< ", type: " << KeysType_Name(tablet->keys_type()) << ", txn_id: " << request.transaction_id
Expand Down

0 comments on commit c320b4c

Please sign in to comment.