BoazC-MSFT opened a new pull request, #9840: URL: https://github.com/apache/arrow-rs/pull/9840
Return a Parquet error when a compact thrift field is expected to be a `bool` but the field header contains a non-bool wire type. This replaces the remaining `bool_val` unwraps in generated thrift readers and the hand-expanded `DataPageHeaderV2` reader. Add regression tests for malformed `DictionaryPageHeader.is_sorted` and `DataPageHeaderV2.is_compressed` headers to ensure corrupt input returns an error instead of panicking. # Which issue does this PR close? - Closes #9839. # Rationale for this change See #9839 # What changes are included in this PR? Proper error handling instead of using `unwrap`. # Are these changes tested? Yes. Added 2 UTs. # Are there any user-facing changes? An error string that can find its way all the way to the end user. -- 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]
