wjones127 commented on PR #34668:
URL: https://github.com/apache/arrow/pull/34668#issuecomment-1480212638
So I found that the DeltaBitPackDecoder will error if the header is missing,
so I think it makes sense for us to also error if it's missing at the beginning
of DeltaLengthByteArray. To accomplish this, I actually deleted the branch, so
it can get to the `DecodeLengths()` which already handles this error:
https://github.com/wjones127/arrow/blob/f59fe0a2267b12069586e7c6b0351c48bf296371/cpp/src/parquet/encoding.cc#L2424
--
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]
