maedhroz commented on code in PR #4902:
URL: https://github.com/apache/cassandra/pull/4902#discussion_r3482769067
##########
src/java/org/apache/cassandra/concurrent/SEPExecutor.java:
##########
@@ -222,10 +224,22 @@ public void maybeExecuteImmediately(Runnable task)
// in this case in particular we are not processing the rest
of the queue anyway, and so
// the work permit may go wasted if we don't immediately
attempt to spawn another worker
maybeSchedule();
+ // nesting with depth > 1 is not supported
+ taskHolder.setImmediateTask(null);
Review Comment:
Should we move this to the start of the `finally` block to make sure that we
can't leak stale state into the virtual table if some kind of exception is
thrown from `maybeSchedule()`?
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
