aokolnychyi commented on issue #74: Add snapshot summary and operation. URL: https://github.com/apache/incubator-iceberg/pull/74#issuecomment-462033821 This change LGTM. The logic doesn't seem to be computationally expensive even though we might need to iterate through all added/removed files. So, I do not think it will significantly impact the write performance but will definitely give benefits later on.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
