tustvold commented on code in PR #3427:
URL: https://github.com/apache/arrow-rs/pull/3427#discussion_r1062272044


##########
arrow-integration-testing/tests/ipc_reader.rs:
##########
@@ -102,163 +95,119 @@ fn read_generated_be_files_should_work() {
         .unwrap();
 
         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
+        //verify_arrow_file(&testdata, "1.0.0-bigendian", path);
     });
 }
 
 #[test]
-fn projection_should_work() {
-    // complementary to the previous test
+fn read_1_0_0_littleendian() {
     let testdata = arrow_test_data();
+    let version = "1.0.0-littleendian";
     let paths = vec![
-        "generated_interval",
         "generated_datetime",
+        "generated_custom_metadata",
+        "generated_decimal",
+        "generated_decimal256",
+        "generated_dictionary",
+        "generated_dictionary_unsigned",
+        "generated_duplicate_fieldnames",
+        "generated_extension",
+        "generated_interval",
         "generated_map",
+        // fails with
+        // thread 'read_1_0_0_littleendian' panicked at 'assertion failed: 
`(left == right)`
+        //"generated_map_non_canonical",

Review Comment:
   I think it might be worth a ticket for this test, it seems to fail in 
multiple places



-- 
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]

Reply via email to