[
https://issues.apache.org/jira/browse/PARQUET-2222?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17701920#comment-17701920
]
ASF GitHub Bot commented on PARQUET-2222:
-----------------------------------------
pitrou commented on PR #193:
URL: https://github.com/apache/parquet-format/pull/193#issuecomment-1474186504
I think this should be more explicit, e.g.:
```
// The length-prepended version is used for:
// - in v1 data pages: definition levels, repetition levels, and RLE-encoded
boolean data
// - in v2 data pages: RLE-encoded boolean data
length-prepended-rle-bit-packed-hybrid: <length> <encoded-data>
// The non-length-prepended version is used for:
// - in v1 data pages: dictionary indices
// - in v2 data pages: definition levels, repetition levels, and dictionary
indices
rle-bit-packed-hybrid: <encoded-data>
length := length of the <encoded-data> in bytes stored as 4 bytes little
endian (unsigned int32)
encoded-data := <run>*
```
> [Format] RLE encoding spec incorrect for v2 data pages
> ------------------------------------------------------
>
> Key: PARQUET-2222
> URL: https://issues.apache.org/jira/browse/PARQUET-2222
> Project: Parquet
> Issue Type: Bug
> Components: parquet-format
> Reporter: Antoine Pitrou
> Priority: Critical
> Fix For: format-2.10.0
>
>
> The spec
> (https://github.com/apache/parquet-format/blob/master/Encodings.md#run-length-encoding--bit-packing-hybrid-rle--3)
> has this:
> {code}
> rle-bit-packed-hybrid: <length> <encoded-data>
> length := length of the <encoded-data> in bytes stored as 4 bytes little
> endian (unsigned int32)
> {code}
> But the length is actually prepended only in v1 data pages, not in v2 data
> pages.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
