munendrasn commented on PR #11548: URL: https://github.com/apache/iceberg/pull/11548#issuecomment-2497943061
@nastra Agreed on having more configuration flags.. The original intent was to enable `useSystemProperties` on HttpClient too as mentioned in the Description. This would help in consuming the Java system properties directly for cases like Http Proxy, or keepLive configuration Reasoning for introducing new configuration property to enable or disable usingSystemProperties : One sample usecase, where we need to connect to Different RESTCatalog impl One of the impl requires mTLS connection and other doesn't. We specify trustStore and keyStore impl via Java System properties We would like to consume these systemProperties in one case but not the other case.. hence, configuration property -- 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-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
