pritampan commented on PR #18055: URL: https://github.com/apache/iceberg/pull/18055#issuecomment-5625838198
Thanks for the review. Renamed the new test and added coverage for `write.delete.parquet.row-group-size-bytes`. Agreed on the operational side: 512 MiB–1 GiB is a more practical row-group target. This change only lifts the `Integer.MAX_VALUE` parse ceiling; the default stays 128 MiB. Related work: I have been experimenting with Parquet layout on my own, and row-group size is one of the levers. I sometimes try a somewhat larger group than the default; hitting the `int` parse limit is what motivated this fix. -- 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]
