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/05 23:28:54 UTC

[GitHub] [druid] mgill25 commented on issue #9286: Prohibit assigning ScheduledExecutorService into ExecutorService

mgill25 commented on issue #9286: Prohibit assigning ScheduledExecutorService into ExecutorService
URL: https://github.com/apache/druid/issues/9286#issuecomment-582662735
 
 
   Hi. I'd like to gently start off contributing to Apache druid, and would love to pick up this issue. 
   
   If I understand correctly, for the two instances in this search, I should make sure that a regular `ExecutorService` is created for the variables instead of `ScheduledExecutorService`. I suspect there are already concurrency utility functions that help in doing that within the codebase that I can use? I'm not at all familiar with the codebase, so I will appreciate any guidance towards how to go on about fixing these issues.
   
   Thank you!

----------------------------------------------------------------
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