priyeshkaratha commented on PR #9413: URL: https://github.com/apache/ozone/pull/9413#issuecomment-3645194730
@devmadhuu Thanks for the detailed review. I had tested some performance benchmark whether we should use our own thread pool with HttpClient or switch to AsyncHttpClient. Based on our performance tests, AsyncHttpClient didn’t provide any real advantage for our use case. So, I reverted those changes and made a few updates based on our discussion. I’ve also addressed your review comments. Performance comparison metric is given below. <img width="1238" height="322" alt="image" src="https://github.com/user-attachments/assets/a8f84f66-f942-49d5-a78a-c696c82ea13c"; /> -- 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]
