ascherbakoff commented on code in PR #7244:
URL: https://github.com/apache/ignite-3/pull/7244#discussion_r2630911967


##########
modules/raft/src/main/java/org/apache/ignite/internal/raft/server/impl/JraftServerImpl.java:
##########
@@ -868,13 +871,17 @@ public void onApply(Iterator iter) {
                     st = new Status(RaftError.ESTATEMACHINE, "Unknown state 
machine error.");
                 }
 
-                // This is necessary so that IndexOutOfBoundsException is not 
thrown in a situation where the listener, when processing a
-                // command, catch any exception and does clo.result(throwable) 
(that actually advances the iterator) and then throws the
-                // caught exception.
-                Closure done = writeCommandIterator.doneForExceptionHandling();
+                if (iterWrapper.done != null) {

Review Comment:
   🆗 



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to