[
https://issues.apache.org/jira/browse/PARQUET-2222?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17732890#comment-17732890
]
ASF GitHub Bot commented on PARQUET-2222:
-----------------------------------------
gszadovszky commented on code in PR #211:
URL: https://github.com/apache/parquet-format/pull/211#discussion_r1230535466
##########
Encodings.md:
##########
@@ -61,7 +61,8 @@ followed by the values encoded using RLE/Bit packed described
above (with the gi
Using the PLAIN_DICTIONARY enum value is deprecated in the Parquet 2.0
specification. Prefer using RLE_DICTIONARY
in a data page and PLAIN in a dictionary page for Parquet 2.0+ files.
-### <a name="RLE"></a>Run Length Encoding / Bit-Packing Hybrid (RLE = 3)
+<a name="RLE"></a>
Review Comment:
Is moving the anchor out of the header an actual fix or just formatting?
Asking only because this is how all the rest of the headers are.
> [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
> Assignee: Gang Wu
> 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)
