You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/02/10 22:34:18 UTC
[GitHub] [druid] jihoonson commented on issue #9325: Use ExecutorService
instead of ScheduledExecutorService where necessary
jihoonson commented on issue #9325: Use ExecutorService instead of ScheduledExecutorService where necessary
URL: https://github.com/apache/druid/pull/9325#issuecomment-584391262
Hi @mgill25, thank you for your contribution. The change looks good to me, but technically this PR doesn’t fix the issue #9286 because it is about prohibiting this pattern by adding a new inspection rule. Are you interested in adding such a rule? If so please include the new rule in the PR. If not, please update the PR description to not include “Fixes #9286”.
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org