gianm opened a new pull request, #19927: URL: https://github.com/apache/druid/pull/19927
`DartWorkerClientImpl` is scoped to a single query, so there is no need to be able to reopen clients (once a worker fails, the query also fails). Prior to this patch, a client would be reopened if requested after being closed. This patch fixes a retry loop that could be caused when a server goes away: `ControllerMessageListener#serverRemoved` calls `closeClient`, but then the call to `stopWorker` would cause the client to be re-created and keep retrying until its retries are exhausted. -- 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]
