[GitHub] [incubator-druid] jon-wei commented on issue #8107: Add CliIndexer process type and initial task runner implementation

[GitBox <gi...@apache.org>]

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-514832272
 
 
   @clintropolis 
   
   I observed the Jetty server shutdown before task shutdown in a Peon too, that may be worth looking into as a separate issue.
   
   The hanging you observed was due to SeekableStreamIndexTask not doing any shutdown actions if `druid.indexer.task.restoreTasksOnRestart=false`, so the indexer just waited for the graceful shutdown timeout before interrupting the task thread.
   
   I've updated the patch to have the stopGracefully() methods on the tasks interrupt their run method

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