etseidl commented on code in PR #9924:
URL: https://github.com/apache/arrow-rs/pull/9924#discussion_r3202897807
##########
parquet/tests/arrow_reader/bad_data.rs:
##########
@@ -98,7 +98,7 @@ fn test_arrow_gh_41317() {
let err = read_file("ARROW-GH-41317.parquet").unwrap_err();
assert_eq!(
err.to_string(),
- "External: Parquet argument error: Parquet error: StructArrayReader
out of sync in read_records, expected 5 read, got 2"
+ "Parquet error: Expected list element type of I32 but got I16"
Review Comment:
My concern is that the file is intended to test a completely different
error. I wonder if it's worth going in with a hex editor to fix the bad list so
the only error is the column mismatch.
##########
parquet/tests/arrow_reader/bad_data.rs:
##########
@@ -98,7 +98,7 @@ fn test_arrow_gh_41317() {
let err = read_file("ARROW-GH-41317.parquet").unwrap_err();
assert_eq!(
err.to_string(),
- "External: Parquet argument error: Parquet error: StructArrayReader
out of sync in read_records, expected 5 read, got 2"
+ "Parquet error: Expected list element type of I32 but got I16"
Review Comment:
My concern is that the file is intended to test a completely different
error. I wonder if it's worth going in with a hex editor to fix the bad list so
the only error is the column mismatch. I think I'll file an issue on
parquet-testing.
--
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]
