rpuch commented on code in PR #4950: URL: https://github.com/apache/ignite-3/pull/4950#discussion_r1895376014
########## modules/raft/src/main/java/org/apache/ignite/raft/jraft/core/IteratorImpl.java: ########## @@ -135,6 +139,19 @@ protected void runTheRestClosureWithError() { } } + void runTheRestClosureWithShutdownException() { + Exception shutdownException = new ShutdownException(); Review Comment: One exception per Raft group doesn't seem to be too much. I'm not sure whether we need a stack trace now, but, even for debugging purposes, it might be useful to have one when this doesn't incur a noticeable performance penalty (as in this case). -- 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: notifications-unsubscr...@ignite.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org