gh-yzou commented on PR #12655: URL: https://github.com/apache/iceberg/pull/12655#issuecomment-2755697834
@adutra Thanks a lot for putting on this PR! Yes, have a way to inject the AuthManager would be very useful for Polaris Spark client. Polaris provides native support for Iceberg APIs, and it also provides set of APIs that is owned by Polaris. The Spark plugin provided will help Spark talk to Polaris service for both Iceberg APIs and Polaris specific APIs, and we want to be able to use the same RESTClient and Authentication to talk to all those endpoints. In that way, it could help us to save the extra token refresh overhead and maximize the reuse at client side. This PR helps providing a way to inject a authManager, which could be helpful for us to reuse the authentication across all endpoints. @nastra I am wondering what do you think about 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: 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
