Skip to content

Commit 671fefa

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

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -792,7 +792,7 @@ void finishAsFailed(Exception failure) {
792792
logger.trace(() -> new ParameterizedMessage("operation failed. action [{}], request [{}]", actionName, request), failure);
793793
listener.onFailure(failure);
794794
} else {
795-
assert false : "finishAsFailed called but operation is already finished";
795+
assert false : new AssertionError("finishAsFailed called but operation is already finished", failure);
796796
}
797797
}
798798

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

0 commit comments

Comments
 (0)