jira-importer commented on issue #542:
URL: https://github.com/apache/curator/issues/542#issuecomment-2604693530

   <i><a 
href="https://issues.apache.org/jira/secure/ViewProfile.jspa?name=cheddar";>cheddar</a>:</i>
   <p>Attaching another patch to fix a slight bug with the changes introduced 
here.</p>
   
   <p>Allowing the executor to queue up tasks has introduced a race condition 
in which.  Specifically, if the PathChildrenCache is set to watch for data 
changes, after the PathChildrenCache has been closed the dataWatcher will stay 
registered on the various paths ('cause ZK is awesome like that) and, when 
changes happen to those paths, the watcher will fire and try to submit 
Runnables to the closed ExecutorService.  This results in 
RejectedExecutionExceptions showing up in the logs, there is actually nothing 
functionally wrong with this other than that it scares people who don't know 
what is going on.</p>
   
   <p>This patch fixes it so that it will not generate those exceptions 
anymore.</p>


-- 
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: dev-unsubscr...@curator.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to