[Backport 2.x] Adding configurable connection count setting for S3 Sync Client (#12028)#12053
Conversation
…search-project#12028) * Adding configurable connection count setting for S3 Sync Client Signed-off-by: Gaurav Bafna <gbbafna@amazon.com> * Address PR Comments Signed-off-by: Gaurav Bafna <gbbafna@amazon.com> * Increase number of connections to 500 Signed-off-by: Gaurav Bafna <gbbafna@amazon.com> --------- Signed-off-by: Gaurav Bafna <gbbafna@amazon.com>
Compatibility status:Checks if related components are compatible with change 208e36e Incompatible componentsIncompatible components: [https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/performance-analyzer-rca.git] Skipped componentsCompatible componentsCompatible components: [https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/alerting.git] |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 2.x #12053 +/- ##
============================================
+ Coverage 71.09% 71.19% +0.10%
- Complexity 59394 59458 +64
============================================
Files 4891 4891
Lines 279974 279982 +8
Branches 41070 41070
============================================
+ Hits 199050 199345 +295
+ Misses 63994 63737 -257
+ Partials 16930 16900 -30 ☔ View full report in Codecov by Sentry. |
Adding configurable connection count setting for S3 Sync Client
Address PR Comments
Increase number of connections to 500
Description
max_sync_connectionsfor sync client's max connections.connection_acquisition_timeoutfor sync client as well . Today it is only used in async client.Related Issues
Resolves #12027
Check List
Public documentation issue/PR createdBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.