alamb commented on PR #9372: URL: https://github.com/apache/arrow-rs/pull/9372#issuecomment-5217749936
Here are some follow ups suggested by claude 1. If the first data page of a column chunk has zero non-null values, build_preset(&[]) fixes the scale at 10^0, so every non-integer value in all later pages of that chunk becomes an exception Since an all-null leading page is a realistic input, this is probably something we should fix (as a follow on PR) 2. parquet-rewrite CLI (parquet/src/bin/parquet-rewrite.rs:147) doesn't expose ALP as an encoding option. -- 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]
