Re: [PR] Core: Add table session cache [iceberg]

rdblue merged PR #8920: URL: https://github.com/apache/iceberg/pull/8920 -- 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.apac

Re: [PR] Core: Add table session cache [iceberg]

nastra closed pull request #8920: Core: Add table session cache URL: https://github.com/apache/iceberg/pull/8920 -- 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,

Re: [PR] Core: Add table session cache [iceberg]

nastra commented on code in PR #8920: URL: https://github.com/apache/iceberg/pull/8920#discussion_r1375905299 ## core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java: ## @@ -939,7 +967,12 @@ private static Cache newSessionCache(Map pr return Caffeine.newBuild

Re: [PR] Core: Add table session cache [iceberg]

rdblue commented on code in PR #8920: URL: https://github.com/apache/iceberg/pull/8920#discussion_r1375308246 ## core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java: ## @@ -839,11 +841,37 @@ private FileIO tableFileIO(SessionContext context, Map config) { }

Re: [PR] Core: Add table session cache [iceberg]

rdblue commented on code in PR #8920: URL: https://github.com/apache/iceberg/pull/8920#discussion_r1375308092 ## core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java: ## @@ -939,7 +967,12 @@ private static Cache newSessionCache(Map pr return Caffeine.newBuild

[PR] Core: Add table session cache [iceberg]

nastra opened a new pull request, #8920: URL: https://github.com/apache/iceberg/pull/8920 The purpose of caching auth session for tables is mainly so that we can stop refreshing the session when it isn't used anymore -- This is an automated message from the Apache Git Service. To respond