jon-wei commented on issue #8107: Add CliIndexer process type and initial task 
runner implementation
URL: https://github.com/apache/incubator-druid/pull/8107#issuecomment-516122590
 
 
   @himanshug 
   
   I changed the interrupt to use future.cancel(true), and added a comment 
about how it's not possible to truly force a hard shutdown there (my intent in 
putting it there was for hypothetical cases where a task's stopGracefully takes 
too long or has a bug and doesn't result in a shutdown, but the task is still 
interruptible)

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

Reply via email to