tusharbhatt7 commented on PR #48468: URL: https://github.com/apache/arrow/pull/48468#issuecomment-3645604303
> ### Rationale for this change > Limit the row group size. > > ### What changes are included in this PR? > Add a new config `parquet::WriterProperties::max_row_group_bytes`. > > ### Are these changes tested? > Yes, add unit test. > > ### Are there any user-facing changes? > Yes, user could use the new config to limit row group size. > > * GitHub Issue: [[C++][Parquet] Add configure to limit the row group size #48467](https://github.com/apache/arrow/issues/48467) Thanks for working on this! Since I'm still new to the Arrow codebase, I reviewed the PR at a high level and it helped me understand how WriterProperties and row group configuration are implemented. I don’t have enough experience yet to provide a full technical review, but the approach looks consistent with the design discussed in the issue. Thanks again for sharing this! -- 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]
