wombatu-kun commented on PR #16565: URL: https://github.com/apache/iceberg/pull/16565#issuecomment-5066618506
@kevinjqliu friendly ping. To your point about SCALE and test semantics: the tests whose assertions actually depend on data volume (size-based splitting, sort and z-order shuffle output, partition change) now use a new LARGE_SCALE constant that keeps the original 400000, so those are unchanged. Only the tests asserting on file/snapshot counts and rewrite structure use the smaller scale. The PR is also rebased onto main as a standalone change now (it had been stacked on #16740, which was reverted in #16912). Could you take another look, or help pull in a Spark reviewer? Thanks! -- 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]
