ashishkumar50 commented on PR #6189: URL: https://github.com/apache/ozone/pull/6189#issuecomment-1935435320
> This code is executed in the OM's request handler when OM receives the request from the client. Client --> OM --> SCM. > > IMO It's probably not a good idea to sleep inside a request handler, because if there are many clients, their request will all get blocked because the thread pool for the request handlers is exhausted, which may cause other requests to fail. > > Would it make more sense to retry at client side? Yes makes sense. Moved retry logic at client side. -- 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]
