mgill25 opened a new pull request #9325: Use ExecutorService instead of ScheduledExecutorService where necessary URL: https://github.com/apache/druid/pull/9325 Fixes #9286 ### Description <!-- Describe your patch: what did you change in code? How did you fix the problem? --> Changed the two instances mentioned in the PR to use `Execs.singleThreaded`, which returns an `ExecutorService`, instead of a `ScheduledExecutorService`. I looked at the concurrency checklist (Threads and Executors section), but don't see anything immediate that I also need to do. Please correct me if I missed something. <hr> This PR has: - [x] been self-reviewed. - [x] using the [concurrency checklist](https://github.com/apache/druid/blob/master/dev/code-review/concurrency.md) (Remove this item if the PR doesn't have any relation to concurrency.) <hr>
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
