harshkha3417 commented on issue #7644: URL: https://github.com/apache/shenyu/issues/7644#issuecomment-6220038474
Hi @Aias00 , I have submitted PR #7684 to address the acquisition queue configuration scope of this issue. ### Summary of Changes: - **Configurable acquisition capacity:** Added `maxPendingAcquires` (along with `pendingAcquireMaxCount` alias) to `HttpClientProperties.Pool`. - **Builder wiring & validation:** Wired it into `HttpClientFactory.buildFixedConnectionPool` and `buildElasticConnectionPool` with validation (`> 0` or `-1`). - **Backward compatibility:** Left default as `null` so that when unspecified, Reactor Netty’s default (`2 * maxConnections`) continues to take effect without any behavior change. - **Config sample:** Updated the commented `httpclient.pool` section in `shenyu-bootstrap/application.yml` to reflect `type: FIXED` and document the new queue setting. - **Tests & Quality:** Added unit tests in `HttpClientPluginConfigurationTest` and `HttpClientPropertiesTest` covering property binding, explicit counts, and boundary checks; verified clean build with 0 Checkstyle violations. Looking forward to your review and feedback! -- 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]
