HoustonPutman opened a new pull request, #3266: URL: https://github.com/apache/solr/pull/3266
https://issues.apache.org/jira/browse/SOLR-17702 This will fix the flaky `org.apache.solr.cloud.BasicDistributedZk2Test` (class method) failures. But in general this is a better way of taking down Solr. There's no reason for SolrZkClient to own this logic anymore. The classes that want to watch on connection state can decide if they want to be async or sync, and how to handle closing the executor themselves. -- 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: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
