apache / parquet-format

Apache Parquet Format
https://parquet.apache.org/
Apache License 2.0
1.76k stars 428 forks source link

PARQUET-2222: Fix broken link for Plain Boolean #211

Closed mapleFU closed 1 year ago

mapleFU commented 1 year ago

Make sure you have checked all steps below.

Jira

https://issues.apache.org/jira/browse/PARQUET-2222

Commits

Documentation

Is already documented

mapleFU commented 1 year ago

@pitrou @gszadovszky @wgtmac

This patch fix the RLE link in PLAIN, and also fix all internal links. It can be seen in https://github.com/mapleFU/parquet-format/blob/parquet/fix-link-for-bitpack/Encodings.md

gszadovszky commented 1 year ago

@mapleFU, could you also fix this?

We still have Data page v1 / Boolean values listed in the prepend length table of RLE which is misleading.

mapleFU commented 1 year ago

@gszadovszky Personally I'd like to fix it in another patch, I'm drafting it currently, this patch just fix the anchor.

gszadovszky commented 1 year ago

@gszadovszky Personally I'd like to fix it in another patch, I'm drafting it currently, this patch just fix the anchor.

@mapleFU, sounds good. Thanks a lot!