Skip to content

Commit

Permalink
Minor: add ticket links to failing ipc integration tests (#3461)
Browse files Browse the repository at this point in the history
  • Loading branch information
alamb authored Jan 5, 2023
1 parent 42ffc3f commit 4f44c2d
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
10 changes: 6 additions & 4 deletions arrow-integration-testing/tests/ipc_reader.rs
Original file line number Diff line number Diff line change
Expand Up @@ -96,8 +96,11 @@ fn read_1_0_0_bigendian() {

FileReader::try_new(file, None).unwrap();

// While the the reader doesn't error but the values are not read correctly
// so verifing the contents fails
// While the the reader doesn't error but the values are not
// read correctly on little endian platforms so verifing the
// contents fails
//
// https://github.com/apache/arrow-rs/issues/3459
//verify_arrow_file(&testdata, "1.0.0-bigendian", path);
});
}
Expand All @@ -117,8 +120,7 @@ fn read_1_0_0_littleendian() {
"generated_extension",
"generated_interval",
"generated_map",
// fails with
// thread 'read_1_0_0_littleendian' panicked at 'assertion failed: `(left == right)`
// https://github.com/apache/arrow-rs/issues/3460
//"generated_map_non_canonical",
"generated_nested",
"generated_nested_dictionary",
Expand Down
2 changes: 1 addition & 1 deletion arrow-integration-testing/tests/ipc_writer.rs
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ fn write_1_0_0_littleendian() {
"generated_extension",
"generated_interval",
"generated_map",
// thread 'write_1_0_0_littleendian' panicked at 'assertion failed: `(left == right)`
// https://github.com/apache/arrow-rs/issues/3460
// "generated_map_non_canonical",
"generated_nested",
"generated_nested_dictionary",
Expand Down

0 comments on commit 4f44c2d

Please sign in to comment.