You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "etseidl (via GitHub)" <gi...@apache.org> on 2023/02/10 19:09:07 UTC

[GitHub] [arrow] etseidl commented on pull request #34096: GH-34086: [C++][Parquet] Fix writing num_rows to data page v2

etseidl commented on PR #34096:
URL: https://github.com/apache/arrow/pull/34096#issuecomment-1426224839

   @mapleFU the issue is with nested columns (a list column for instance).  The number of values in a given page for that column will be different from the number of rows.  The previous writer was using `num_values` for both the `num_values` and `num_rows` fields in the page header.  This PR fixes this to use `num_rows` for the `num_rows` field.


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org