rschmitt commented on pull request #296: URL: https://github.com/apache/httpcomponents-client/pull/296#issuecomment-803086069
It seems that `RequestConfig` and `ConnectionMgmtConfig` are always passed together and both specify per-request options. In that case, why not declare `ConnectionMgmtConfig` _on_ `RequestConfig`? In addition to simplifying the diff, it will make it clear that `ConnectionMgmtConfig` are all request-scoped (so to speak). -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
