YongGang commented on PR #14898: URL: https://github.com/apache/druid/pull/14898#issuecomment-1694757685
> > i think we may have found a solution here that is simpler, to move some logic from restore() (which is called asyncronously in Task.manage) to start (which is called during LifecycleStart of the taskRunner before SupervisorManager starts) > > Great! I am not sure I fully understand the solution, but if it works, I would be happy to review the PR. Hi @kfaraz , this is the PR to fix the original issue https://github.com/apache/druid/pull/14909 But here I still like to address the leader query problem (maybe not in the way I proposed, that's why it's submitted as draft for discussion). I will think more about it. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
