gaborkaszab commented on PR #14398: URL: https://github.com/apache/iceberg/pull/14398#issuecomment-3657399909
Update on the latest changes: There have been multiple merged PRs recently that conflicted with this one: injectable RESTTableOperations and server-side planning. In fact because of these features the freshness-aware loading code had to be revised to support interworking with the mentioned functionalities. As a result, the table cache doesn't hold actual table objects, but table suppliers, that can be used to produce table objects in case of a 304. With this, encapsulating TableOps and RESTTable creation into this table supplier we won't break injectable ops, and we can use freshness-aware loading for RESTTables too. Thanks @pvary for the help here! As a side-effect, since we don't cache table (including ops and IO), the FileIOTracker tests are no longer needed within this PR. Removed them. -- 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]
