tusharbhatt7 commented on issue #48467:
URL: https://github.com/apache/arrow/issues/48467#issuecomment-3645269266

   Hi, I’d like to work on this issue.
   
   To confirm the expected behavior: the idea is to add a new ParquetWriter 
property (similar to parquet-mr’s `maxRowGroupSize`) that limits the *byte 
size* of a row group, complementing the existing `max_row_group_length` (which 
limits the number of rows).
   
   My plan:
   1. Add a new optional config field in `parquet::WriterProperties` for max 
row group size in bytes.
   2. Enforce this condition in `parquet::RowGroupWriter` so a new row group is 
started once the accumulated buffered size exceeds the limit.
   3. Mirror semantics from parquet-mr implementation.
   4. Add unit tests verifying correct splits for variable-size columns.
   5. Update documentation.
   
   Please let me know if this approach matches what you had in mind or if you’d 
prefer a specific naming/config structure.
   Thanks — Tushar


-- 
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]

Reply via email to