Skip to content

Commit 694263a

Browse files
committed
Format code and update comment
1 parent b317928 commit 694263a

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

fdbclient/SpecialKeySpace.actor.cpp

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1013,9 +1013,8 @@ ACTOR Future<bool> checkExclusion(Database db,
10131013
return false;
10141014
}
10151015
NetworkAddress addr = NetworkAddress::parse(addrStr);
1016-
bool includedInExclusion = addressExcluded(*exclusions, addr);
1017-
bool excluded =
1018-
(process.has("excluded") && process.last().get_bool()) || includedInExclusion;
1016+
bool includedInExclusion = addressExcluded(*exclusions, addr);
1017+
bool excluded = (process.has("excluded") && process.last().get_bool()) || includedInExclusion;
10191018

10201019
StatusObjectReader localityObj;
10211020
std::string disk_id;
@@ -1028,9 +1027,8 @@ ACTOR Future<bool> checkExclusion(Database db,
10281027
if (role["role"].get_str() == "storage") {
10291028
ssTotalCount++;
10301029

1031-
// Check if we are excluding a process that serves the storage role. If this check was true once, we
1032-
// don't have to check any further since we don't case in this variable about the count of excluded
1033-
// storage servers but only about if we exclude any storage server with the provided addresses.
1030+
// Check if we are excluding a process that serves the storage role. We only have to check the free
1031+
// capacity if we are excluding at least one process that serves the storage role.
10341032
if (!excludedAddressesContainsStorageRole && includedInExclusion) {
10351033
excludedAddressesContainsStorageRole = true;
10361034
break;

0 commit comments

Comments
 (0)