xuzifu666 commented on code in PR #9725:
URL: https://github.com/apache/arrow-rs/pull/9725#discussion_r3093595938
##########
parquet/src/file/metadata/reader.rs:
##########
@@ -1422,4 +1434,81 @@ mod async_tests {
read_and_check(f.as_file(), PageIndexPolicy::Optional).unwrap();
read_and_check(f.as_file(), PageIndexPolicy::Skip).unwrap();
}
+
+ /// Test that corrupted parquet files return ParquetError instead of
panicking
Review Comment:
Thanks for your suggestion! Previous test way is not suitable, I had
deleted it. Can we resolve this issue in this way for now? Parquet fuzz tsting
could be a follow-up task~ @alamb
--
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]
