[CURATOR-557] ServiceCacheImpl does not close ExecutorService

ServiceCacheImpl does not close the ExecutorService instance created from the ThreadFactory:

https://github.com/apache/curator/blob/master/curator-x-discovery/src/main/java/org/apache/curator/x/discovery/details/ServiceCacheImpl.java#L64

CloseableExecutorService::CloseableExecutorService(ExecutorService) call this(executorService, false) which sets shutdownOnClose to false.

this has an impact from callers, eg ServiceProviderImpl, which only allow only a ThreadFactory to be specified.

---
Originally reported by rnaude, imported from: ServiceCacheImpl does not close ExecutorService

• status: Open
  
• priority: Major
  
• resolution: Unresolved
  
• imported: 2025-01-21