danielcweeks merged PR #9195:
URL: https://github.com/apache/iceberg/pull/9195
--
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:
danielcweeks commented on code in PR #9195:
URL: https://github.com/apache/iceberg/pull/9195#discussion_r1412877063
##
core/src/main/java/org/apache/iceberg/rest/HTTPClient.java:
##
@@ -72,6 +74,10 @@ public class HTTPClient implements RESTClient {
static final String
amogh-jahagirdar commented on code in PR #9195:
URL: https://github.com/apache/iceberg/pull/9195#discussion_r1412727587
##
core/src/main/java/org/apache/iceberg/rest/HTTPClient.java:
##
@@ -72,6 +74,10 @@ public class HTTPClient implements RESTClient {
static final String
amogh-jahagirdar commented on code in PR #9195:
URL: https://github.com/apache/iceberg/pull/9195#discussion_r1412727587
##
core/src/main/java/org/apache/iceberg/rest/HTTPClient.java:
##
@@ -72,6 +74,10 @@ public class HTTPClient implements RESTClient {
static final String
danielcweeks opened a new pull request, #9195:
URL: https://github.com/apache/iceberg/pull/9195
The default config for http connections is only 25 connections and 5 per
host. Since the REST client is typically talking to a single host, the number
of connections allowed is relatively small