gaborkaszab commented on PR #14166: URL: https://github.com/apache/iceberg/pull/14166#issuecomment-3784230863
Hi All, I've been active on this part of the code recently, and I think this PR makes much sense. If the catalog is in REFS mode then the refresh() on the table should also follow this strategy. E.g. table is loaded with REFS only, then a refresh shouldn't load all the snapshots. What could also be misleading that these 2 use-cases work differently now: 1) table loaded with REFS, there are no changes since load -> refresh won't load the rest of the snapshots (because metadata location is the same) 2) table loaded with REFS, there are some changes since load -> refresh will load all snapshots not just refs. @gabeiglio do you think we can revive this PR? If you have no capacity ATM, I can take over, let me know! @nastra Do you think you can take a look once revived? -- 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]
