kbendick commented on issue #5442: URL: https://github.com/apache/iceberg/issues/5442#issuecomment-1206814006
Oh this seems intentional based on the comment above `newSnapshotLog.clear()` below. Though there's possibly a better way of handling the intermediate history states that never actually existed vs simply clearing the whole history log. I think the `snapshots` metadata table would still have all the snapshot data, but ideally just the entries that need to be trimmed from the history log are removed. https://github.com/apache/iceberg/blob/353c53910195b81bd5ceb908c12ed8113bbb4e3b/core/src/main/java/org/apache/iceberg/TableMetadata.java#L1558-L1593 -- 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]
