ajantha-bhat commented on PR #9455: URL: https://github.com/apache/iceberg/pull/9455#issuecomment-1893176190
> the only thing I'm a little concerned about is the CachingCatalog, since it uses only the TableIdentifier as the cache key and doesn't consider the snapshot id of a table. That means you could get wrong results back @nastra: Caching catalog is only used to get the table object using identifier and snpshot id will be overriden for time travel queries right? Else the time travel queries will not work. So, I think it should be ok. WDYT? -- 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
