Skip to content

Commit

Permalink
Remove snapshot creation when add_member done
Browse files Browse the repository at this point in the history
Nuraft Reconfigure issue has been fixed, we don't need to create snapshot.
  • Loading branch information
yuwmao authored and xiaoxichen committed Feb 12, 2025
1 parent 3e38fa8 commit 3a34040
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 10 deletions.
2 changes: 1 addition & 1 deletion conanfile.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@

class HomestoreConan(ConanFile):
name = "homestore"
version = "6.6.17"
version = "6.6.18"

homepage = "https://github.com/eBay/Homestore"
description = "HomeStore Storage Engine"
Expand Down
9 changes: 0 additions & 9 deletions src/lib/replication/repl_dev/raft_repl_dev.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -178,15 +178,6 @@ AsyncReplResult<> RaftReplDev::replace_member(const replica_member_info& member_
RD_LOGI("Replace member added member={} to group_id={}", boost::uuids::to_string(member_in.id),
group_id_str());

// If enabled, create a snapshot here to ensure the new member will use the latest snapshot with itself in the config
if (raft_server()->get_current_params().snapshot_distance_ > 0) {
if (auto idx = raft_server()->create_snapshot(); idx > 0) {
RD_LOGI("Created snapshot idx={} after adding member", idx);
} else {
RD_LOGW("Failed to create snapshot after adding member");
}
}

// Step 3. Append log entry to mark the old member is out and new member is added.
auto rreq = repl_req_ptr_t(new repl_req_ctx{});
replace_members_ctx members;
Expand Down

0 comments on commit 3a34040

Please sign in to comment.