Skip to content

Commit e19585b

Browse files
Enhance TransportReplicationAction assertions (#49081)
Include failure into assertion error when replication action discovers that it has been double triggered.
1 parent 338dd64 commit e19585b

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

server/src/main/java/org/elasticsearch/action/support/replication/TransportReplicationAction.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -793,7 +793,7 @@ void finishAsFailed(Exception failure) {
793793
logger.trace(() -> new ParameterizedMessage("operation failed. action [{}], request [{}]", actionName, request), failure);
794794
listener.onFailure(failure);
795795
} else {
796-
assert false : "finishAsFailed called but operation is already finished";
796+
assert false : new AssertionError("finishAsFailed called but operation is already finished", failure);
797797
}
798798
}
799799

@@ -805,7 +805,7 @@ void finishWithUnexpectedFailure(Exception failure) {
805805
setPhase(task, "failed");
806806
listener.onFailure(failure);
807807
} else {
808-
assert false : "finishWithUnexpectedFailure called but operation is already finished";
808+
assert false : new AssertionError("finishWithUnexpectedFailure called but operation is already finished", failure);
809809
}
810810
}
811811

0 commit comments

Comments
 (0)