Xuanwo opened a new pull request, #1221:
URL: https://github.com/apache/iceberg-rust/pull/1221

   ## Which issue does this PR close?
   
   Creating a reqwest client is not without cost; it involves DNS caching, HTTP 
connection pooling, and TLS certificate loading. The previous design also made 
it difficult for users to configure the HTTP client, such as setting custom 
timeout values or using a globally shared DNS resolver.
   
   This PR addressed this by allowing users to create the client and pass to 
rest catalog as config.
   
   ## What changes are included in this PR?
   
   Allow users to pass a `reqwest::Client` as config.
   
   ## Are these changes tested?
   
   Unit tests


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

Reply via email to