FastRestore:Revise sanity check condition
This commit is contained in:
parent
52e3d20d39
commit
c4a11597cf
|
@ -419,16 +419,15 @@ ACTOR static Future<Void> sendMutationsFromLoaders(Reference<MasterBatchData> ba
|
||||||
|
|
||||||
for (auto& reply : replies) {
|
for (auto& reply : replies) {
|
||||||
RestoreSendStatus status = batchStatus->loadStatus[reply.id];
|
RestoreSendStatus status = batchStatus->loadStatus[reply.id];
|
||||||
if ((status == RestoreSendStatus::SendingRanges || status == RestoreSendStatus::SendingLogs) &&
|
if ((status == RestoreSendStatus::SendingRanges || status == RestoreSendStatus::SendingLogs)) {
|
||||||
!reply.isDuplicated) {
|
|
||||||
batchStatus->loadStatus[reply.id] = (status == RestoreSendStatus::SendingRanges)
|
batchStatus->loadStatus[reply.id] = (status == RestoreSendStatus::SendingRanges)
|
||||||
? RestoreSendStatus::SendedRanges
|
? RestoreSendStatus::SendedRanges
|
||||||
: RestoreSendStatus::SendedLogs;
|
: RestoreSendStatus::SendedLogs;
|
||||||
} else if ((status == RestoreSendStatus::SendingRanges || status == RestoreSendStatus::SendingLogs) &&
|
if (reply.isDuplicated) {
|
||||||
reply.isDuplicated) {
|
TraceEvent(SevWarn, "FastRestoreSendMutations")
|
||||||
TraceEvent(SevWarn, "FastRestoreSendMutations")
|
|
||||||
.detail("Loader", reply.id)
|
.detail("Loader", reply.id)
|
||||||
.detail("RequestUnprocessed", "Waiting for ack that loader has processed the request");
|
.detail("DuplicateRequestAcked", "Request should have been processed");
|
||||||
|
}
|
||||||
} else if ((status == RestoreSendStatus::SendedRanges || status == RestoreSendStatus::SendedLogs) &&
|
} else if ((status == RestoreSendStatus::SendedRanges || status == RestoreSendStatus::SendedLogs) &&
|
||||||
reply.isDuplicated) {
|
reply.isDuplicated) {
|
||||||
TraceEvent(SevDebug, "FastRestoreSendMutations")
|
TraceEvent(SevDebug, "FastRestoreSendMutations")
|
||||||
|
|
Loading…
Reference in New Issue