Neuw84 commented on PR #18258: URL: https://github.com/apache/iceberg/pull/18258#issuecomment-6087577803
Small follow-up to my previous comment: I've reverted the hashed key in c78e1d3 and kept the plain one. As you said, building the key is negligible next to building the delete set, and with merged-set caching now opt-in your main concern, partially overlapping delete file sets, is covered without it. That keeps the PR smaller. -- 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]
