gianm opened a new pull request #11956: URL: https://github.com/apache/druid/pull/11956
Usually, "execute" is called by methods defined in the superclass AbstractExecutorService, and the passed-in Runnable has been wrapped by newTaskFor inside a PrioritizedListenableFutureTask. But this method can also be called directly, and if so, the same wrapping is necessary for the delegate to get a Runnable that can be entered into a priority queue with the others. I don't think this affects production, because I didn't find any production code that calls this method. I noticed it while working on another branch doing something that _did_ end up calling this method on a PrioritizedExecutorService, so I thought it would be nice to split the fix into its own patch and test. -- 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]
