Re: [PR] Read the max connection for rest client from properties [iceberg]

["munendrasn (via GitHub)" <gi...@apache.org>]

munendrasn commented on PR #11522:
URL: https://github.com/apache/iceberg/pull/11522#issuecomment-2470504796

   @danielcweeks Please review
   
   Currently, max Connections are read from System property. Looking through the code for both other similar properties, this property behaviour is different. 
   Updated to read from the properties passed to httpClient builder.
   
   Please let me know if support to read from System property is intended or required for backward compatibility 
   


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org