danielcweeks commented on PR #12868: URL: https://github.com/apache/iceberg/pull/12868#issuecomment-3319697615
> Currently, the mapping is already Table -> S3FileIO -> S3Client -> HttpClient. (1 to 1) The problem is that multiple HttpClient instances can still end up sharing the same underlying connection pool. @xiaoxuandev I don't believe the issue is that we're sharing an underlying client, but rather how the SDK is interacting with it. My understanding is that if we change how we configure the S3 Client using the approach I mention above, then it should be properly reused and not closed when the S3FileIO or S3 Client are closed. I don't think the proposed approach in this PR is the right way to address this. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
