bahman/scylla
1
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

raft topology: don't include null ID in exclude_nodes

Browse Source 
Observed with newly added logs:
```
raft topology - executing global topology command barrier_and_drain, excluded nodes: {00000000-0000-0000-0000-000000000000}
```
This commit is contained in:
Kamil Braun
2024-01-11 15:42:48 +01:00
parent ae25f703c4
commit aeb53ea31d
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
service/storage_service.cc
View File

@@ -1563,7 +1563,9 @@ class topology_coordinator : public endpoint_lifecycle_subscriber {
std::unordered_set<raft::server_id> get_excluded_nodes(raft::server_id id, const std::optional<topology_request>& req, const std::optional<request_param>& req_param) {
auto exclude_nodes = parse_ignore_nodes(req_param);
exclude_nodes.insert(parse_replaced_node(req_param));
if (auto replaced_node = parse_replaced_node(req_param)) {
exclude_nodes.insert(replaced_node);
}
if (req && *req == topology_request::remove) {
exclude_nodes.insert(id);
}