kfaraz commented on PR #17509: URL: https://github.com/apache/druid/pull/17509#issuecomment-2535975907
> its about a minute which i think should be enough in most cases Looking at `RetryUtils`, I think the total time for 10 retries would be more than that, probably closer to 5 minutes or so. > technically the previous approach also relied on client exponential backoff so we should be talking about a similar amount of time if the retry logic is the same I think the retry logic is different since `ServiceClientImpl` uses the `StandardRetryPolicy` where the default exponential backoff leads to a total retry time of 1 hour approx. That said, I prefer the retry at the metadata level since it is quicker and simpler. -- 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]
