harshkha3417 opened a new pull request, #7684:
URL: https://github.com/apache/shenyu/pull/7684

   ## What is the purpose of the change
   Fixes #7644
   
   Make outbound HTTP connection-pool acquisition capacity explicit and 
configurable.
   Exposes `maxPendingAcquires` (with `pendingAcquireMaxCount` alias) in 
`HttpClientProperties.Pool` and wires it to 
`ConnectionProvider.Builder#pendingAcquireMaxCount` in `HttpClientFactory`.
   
   When unconfigured, Reactor Netty's default (`2 * maxConnections`) remains 
intact for backward compatibility. Also updates the sample connection pool 
configuration in `shenyu-bootstrap/src/main/resources/application.yml` to 
reflect `type: FIXED` and the new acquisition queue setting.
   
   ## Brief changelog
   - Added `maxPendingAcquires` and `pendingAcquireMaxCount` to 
`HttpClientProperties.Pool`.
   - Configured and validated `pendingAcquireMaxCount` in 
`HttpClientFactory.buildFixedConnectionPool` and `buildElasticConnectionPool`.
   - Updated commented `httpclient.pool` section in 
`shenyu-bootstrap/src/main/resources/application.yml`.
   - Added unit tests for property binding, default behavior, explicit 
acquisition counts, and boundary validation in `HttpClientPropertiesTest` and 
`HttpClientPluginConfigurationTest`.
   
   ## Verifying this change
   - `mvn checkstyle:check` passed with 0 violations across all modules.
   - All unit tests in `shenyu-plugin-httpclient` and 
`shenyu-spring-boot-starter-plugin-httpclient` passed.


-- 
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]

Reply via email to