wgtmac opened a new pull request, #34096:
URL: https://github.com/apache/arrow/pull/34096

   ### Rationale for this change
   
   The C++ parquet writer does not correctly fill num_rows field to DataPageV2 
header.
   
   ### What changes are included in this PR?
   
   ColumnWriter keeps track of number of rows buffered in the current data page 
and then fills it into header of data page v2.
   
   ### Are these changes tested?
   
   A test case has been added to make sure the data page header has been set 
correctly for required, optional and repeated columns.
   
   ### Are there any user-facing changes?
   
   No.


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