Merge pull request #6927 from sfc-gh-xwang/main
Run completeDest after clear destination set
This commit is contained in:
commit
b8fbd18199
|
@ -386,15 +386,19 @@ void launchDest(RelocateData& relocation,
|
|||
}
|
||||
}
|
||||
|
||||
void completeDest(RelocateData const& relocation, std::map<UID, Busyness>& destBusymap) {
|
||||
int destWorkFactor = getDestWorkFactor();
|
||||
for (UID id : relocation.completeDests) {
|
||||
destBusymap[id].removeWork(relocation.priority, destWorkFactor);
|
||||
}
|
||||
}
|
||||
|
||||
void complete(RelocateData const& relocation, std::map<UID, Busyness>& busymap, std::map<UID, Busyness>& destBusymap) {
|
||||
ASSERT(relocation.workFactor > 0);
|
||||
for (int i = 0; i < relocation.src.size(); i++)
|
||||
busymap[relocation.src[i]].removeWork(relocation.priority, relocation.workFactor);
|
||||
|
||||
int destWorkFactor = getDestWorkFactor();
|
||||
for (UID id : relocation.completeDests) {
|
||||
destBusymap[id].removeWork(relocation.priority, destWorkFactor);
|
||||
}
|
||||
completeDest(relocation, destBusymap);
|
||||
}
|
||||
|
||||
ACTOR Future<Void> dataDistributionRelocator(struct DDQueueData* self,
|
||||
|
@ -1389,6 +1393,8 @@ ACTOR Future<Void> dataDistributionRelocator(DDQueueData* self, RelocateData rd,
|
|||
} else {
|
||||
TEST(true); // move to removed server
|
||||
healthyDestinations.addDataInFlightToTeam(-metrics.bytes);
|
||||
|
||||
completeDest(rd, self->destBusymap);
|
||||
rd.completeDests.clear();
|
||||
wait(delay(SERVER_KNOBS->RETRY_RELOCATESHARD_DELAY, TaskPriority::DataDistributionLaunch));
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue