arturobernalg commented on PR #592: URL: https://github.com/apache/httpcomponents-core/pull/592#issuecomment-3697165770
> @arturobernalg I do not like this approach. There is already a command queue built into the `IOSession`. We should leverage that queue instead of creating another layer of complexity. What we need is to extend `IOSession` interface to expose information about already pending commands and optionally reject commands if the queue is over a particular limit. @ok2c . I dropped the extra wrapper and enforce the cap via the existing `IOSession` command queue. I extended `IOSession` with a bounded enqueue + pending count and delegated through `InternalDataChannel` and `SSLIOSession` so the limit is not bypassed by decorators. -- 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]
