Apply Clang format to PRIORITY_TEAM_REDUNDANT
This commit is contained in:
parent
ea76451f15
commit
612a51fe00
|
@ -382,14 +382,14 @@ struct DDQueueData {
|
|||
// ensure a team remover will not start before the previous one finishes removing a team and move away data
|
||||
// NOTE: split and merge shard have higher priority. If they have to wait for unhealthyRelocations = 0,
|
||||
// deadlock may happen: split/merge shard waits for unhealthyRelocations, while blocks team_redundant.
|
||||
if(priority >= PRIORITY_TEAM_UNHEALTHY || priority == PRIORITY_TEAM_REDUNDANT) {
|
||||
if (priority >= PRIORITY_TEAM_UNHEALTHY || priority == PRIORITY_TEAM_REDUNDANT) {
|
||||
unhealthyRelocations++;
|
||||
rawProcessingUnhealthy->set(true);
|
||||
}
|
||||
priority_relocations[priority]++;
|
||||
}
|
||||
void finishRelocation(int priority) {
|
||||
if(priority >= PRIORITY_TEAM_UNHEALTHY || priority == PRIORITY_TEAM_REDUNDANT) {
|
||||
if (priority >= PRIORITY_TEAM_UNHEALTHY || priority == PRIORITY_TEAM_REDUNDANT) {
|
||||
unhealthyRelocations--;
|
||||
ASSERT(unhealthyRelocations >= 0);
|
||||
if(unhealthyRelocations == 0) {
|
||||
|
@ -600,7 +600,8 @@ struct DDQueueData {
|
|||
if( foundActiveFetching || foundActiveRelocation ) {
|
||||
rd.wantsNewServers |= rrs.wantsNewServers;
|
||||
rd.startTime = std::min( rd.startTime, rrs.startTime );
|
||||
if( (rrs.priority >= PRIORITY_TEAM_UNHEALTHY || rrs.priority == PRIORITY_TEAM_REDUNDANT) && rd.changesBoundaries() )
|
||||
if ((rrs.priority >= PRIORITY_TEAM_UNHEALTHY || rrs.priority == PRIORITY_TEAM_REDUNDANT) &&
|
||||
rd.changesBoundaries())
|
||||
rd.priority = std::max( rd.priority, rrs.priority );
|
||||
}
|
||||
|
||||
|
@ -759,11 +760,9 @@ struct DDQueueData {
|
|||
bool overlappingInFlight = false;
|
||||
auto intersectingInFlight = inFlight.intersectingRanges( rd.keys );
|
||||
for(auto it = intersectingInFlight.begin(); it != intersectingInFlight.end(); ++it) {
|
||||
if(fetchKeysComplete.count( it->value() ) &&
|
||||
inFlightActors.liveActorAt( it->range().begin ) &&
|
||||
!rd.keys.contains( it->range() ) &&
|
||||
it->value().priority >= rd.priority &&
|
||||
(rd.priority < PRIORITY_TEAM_UNHEALTHY || rd.priority == PRIORITY_TEAM_REDUNDANT)) {
|
||||
if (fetchKeysComplete.count(it->value()) && inFlightActors.liveActorAt(it->range().begin) &&
|
||||
!rd.keys.contains(it->range()) && it->value().priority >= rd.priority &&
|
||||
(rd.priority < PRIORITY_TEAM_UNHEALTHY || rd.priority == PRIORITY_TEAM_REDUNDANT)) {
|
||||
/*TraceEvent("OverlappingInFlight", distributorId)
|
||||
.detail("KeyBegin", it->value().keys.begin)
|
||||
.detail("KeyEnd", it->value().keys.end)
|
||||
|
|
|
@ -1223,10 +1223,10 @@ ACTOR static Future<JsonBuilderObject> dataStatusFetcher(WorkerDetails ddWorker,
|
|||
|
||||
bool primary = inFlight.getInt("Primary");
|
||||
int highestPriority = inFlight.getInt("HighestPriority");
|
||||
|
||||
if(movingHighestPriority < PRIORITY_TEAM_UNHEALTHY) {
|
||||
|
||||
if (movingHighestPriority < PRIORITY_TEAM_UNHEALTHY) {
|
||||
highestPriority = movingHighestPriority;
|
||||
} else if(partitionsInFlight > 0) {
|
||||
} else if (partitionsInFlight > 0) {
|
||||
highestPriority = std::max<int>(highestPriority, PRIORITY_MERGE_SHARD);
|
||||
}
|
||||
|
||||
|
@ -1267,32 +1267,26 @@ ACTOR static Future<JsonBuilderObject> dataStatusFetcher(WorkerDetails ddWorker,
|
|||
stateSectionObj["healthy"] = false;
|
||||
stateSectionObj["name"] = "healing";
|
||||
stateSectionObj["description"] = "Restoring replication factor";
|
||||
}
|
||||
else if (highestPriority >= PRIORITY_MERGE_SHARD) {
|
||||
} else if (highestPriority >= PRIORITY_MERGE_SHARD) {
|
||||
stateSectionObj["healthy"] = true;
|
||||
stateSectionObj["name"] = "healthy_repartitioning";
|
||||
stateSectionObj["description"] = "Repartitioning.";
|
||||
}
|
||||
else if (highestPriority >= PRIORITY_TEAM_REDUNDANT) {
|
||||
} else if (highestPriority >= PRIORITY_TEAM_REDUNDANT) {
|
||||
stateSectionObj["healthy"] = true;
|
||||
stateSectionObj["name"] = "optimizing_team_collections";
|
||||
stateSectionObj["description"] = "Optimizing team collections";
|
||||
}
|
||||
else if (highestPriority >= PRIORITY_TEAM_CONTAINS_UNDESIRED_SERVER) {
|
||||
} else if (highestPriority >= PRIORITY_TEAM_CONTAINS_UNDESIRED_SERVER) {
|
||||
stateSectionObj["healthy"] = true;
|
||||
stateSectionObj["name"] = "healthy_removing_server";
|
||||
stateSectionObj["description"] = "Removing storage server";
|
||||
}
|
||||
else if (highestPriority == PRIORITY_TEAM_HEALTHY) {
|
||||
stateSectionObj["healthy"] = true;
|
||||
} else if (highestPriority == PRIORITY_TEAM_HEALTHY) {
|
||||
stateSectionObj["healthy"] = true;
|
||||
stateSectionObj["name"] = "healthy";
|
||||
}
|
||||
else if (highestPriority >= PRIORITY_REBALANCE_SHARD) {
|
||||
} else if (highestPriority >= PRIORITY_REBALANCE_SHARD) {
|
||||
stateSectionObj["healthy"] = true;
|
||||
stateSectionObj["name"] = "healthy_rebalancing";
|
||||
stateSectionObj["description"] = "Rebalancing";
|
||||
}
|
||||
else if (highestPriority >= 0) {
|
||||
} else if (highestPriority >= 0) {
|
||||
stateSectionObj["healthy"] = true;
|
||||
stateSectionObj["name"] = "healthy";
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue