sanghwa-min commented on PR #714: URL: https://github.com/apache/httpcomponents-client/pull/714#issuecomment-3227626914
@ok2c cc. @arturobernalg The default sleepTime is now set to `maxIdleTime / 10`, while maxIdleTime remains untouched. This brings the maximum survival boundary for an idle connection to `1.1 * maxIdleTime` plus the eviction execution time. Choosing the right divisor is a trade-off between guaranteeing the idle timeout and CPU usage—ideally it would be near zero, but that's not feasible. I'm not sure whether this value should be a fixed default or a new configurable field `sleepTime`, since users can instantiate `IdleConnectionEvictor` directly for full control. If you think making the divisor adjustable would be better, please let me know. -- 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]
