anujmodi2021 commented on PR #5881: URL: https://github.com/apache/hadoop/pull/5881#issuecomment-1794640767
@steveloughran There was a PR from @snvijaya that was pending from a long time and this PR here has a dependency on that PR. Sneha's Pending PR: https://github.com/apache/hadoop/pull/3662 Here the Connection Timeout value along with read timeout was made configurable. In my PR we need this functionality because default 30seconds for Connection timeout is very high and should be configurable with proper default value. We have done some scale workload testing where we have observed that for connection timeout along with using a Static Retry Policy decreasing the default connection timeout from 30s to 2s increases the performance. Also waiting for 30s to establish connection where something is actually with network doesn't make sense. Its better to fail faster and retry faster. Let me know if you have any thoughts on this default value. -- 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]
