[GitHub] [iceberg] rdblue commented on pull request #5125: Reuse rest client for catalog and table operations

[GitBox <gi...@apache.org>]

rdblue commented on PR #5125:
URL: https://github.com/apache/iceberg/pull/5125#issuecomment-1164719475

   This looks good to me.
   
   For anyone looking at this later, the context is that we no longer need to create a client per table now that the Authorization HTTP header is handled by sessions rather than by client. That was the purpose of adding sessions, since we don't need to create a different client (and connection pool) for every auth session.


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