licl2014 commented on issue #8445: In DirectDruidClient, Future cancellation listener shouldn't run in HTTP library thread URL: https://github.com/apache/incubator-druid/issues/8445#issuecomment-532655119 I also have a problem about `cancel query`, the scenario is as follows: 1. I execute a big query which will runs 1 minute .the timeout is default 5 minutes, after it runs 3 second, we cancel it through `curl -X DELETE "http://broker:8082/druid/v2/queryid"` and it will be canceled success and all historical nodes will cancel it success too. 2. but if it runs more than 10 secnods , after we cancel it with the DELETE API, we find the query will be canceled ,but not all historical nodes cancel it too. for 2, after DELETE ,we find some `futures` removed from `queries` in QueryManager ,so `cancelQuery` can't cancel these `futures` .
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
