Enhance TransportReplicationAction assertions (#49081)

Include failure into assertion error when replication action discovers
that it has been double triggered.
This commit is contained in:
Henning Andersen 2020-01-02 19:19:52 +01:00 committed by Henning Andersen
parent 338dd642c4
commit e19585b47f

View File

@ -793,7 +793,7 @@ public abstract class TransportReplicationAction<
logger.trace(() -> new ParameterizedMessage("operation failed. action [{}], request [{}]", actionName, request), failure);
listener.onFailure(failure);
} else {
assert false : "finishAsFailed called but operation is already finished";
assert false : new AssertionError("finishAsFailed called but operation is already finished", failure);
}
}
@ -805,7 +805,7 @@ public abstract class TransportReplicationAction<
setPhase(task, "failed");
listener.onFailure(failure);
} else {
assert false : "finishWithUnexpectedFailure called but operation is already finished";
assert false : new AssertionError("finishWithUnexpectedFailure called but operation is already finished", failure);
}
}