essobedo commented on code in PR #9344:
URL: https://github.com/apache/camel/pull/9344#discussion_r1105933476
##########
core/camel-support/src/main/java/org/apache/camel/saga/InMemorySagaCoordinator.java:
##########
@@ -75,6 +75,13 @@ public String getId() {
@Override
public CompletableFuture<Void> beginStep(Exchange exchange, CamelSagaStep
step) {
+ Status status = currentStatus.get();
+ if (status != Status.RUNNING) {
+ CompletableFuture<Void> res = new CompletableFuture<>();
+ res.completeExceptionally(new IllegalStateException("Cannot begin:
status is " + status));
+ return res;
+ }
+
Review Comment:
Would you mind adding a test case somewhere in
https://github.com/apache/camel/blob/main/core/camel-core/src/test/java/org/apache/camel/processor
to prove that it fixes the bug?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
