alamb commented on code in PR #7096:
URL: https://github.com/apache/arrow-rs/pull/7096#discussion_r1947225752


##########
arrow-ipc/src/reader.rs:
##########
@@ -2492,4 +2539,109 @@ mod tests {
                 assert_eq!(decoded_batch.expect("Failed to read RecordBatch"), 
batch);
             });
     }
+
+    #[test]
+    fn test_validation_of_invalid_list_array() {

Review Comment:
   One thing I tried to do was make null buffers that had different sizes than 
the underlying values -- that actually failed the ArrayData creation checks, 
but when it was re-read via IPC the resulting arrays were fine.
   
   Maybe it was due to padding or something. I'll play around with it some more
   



-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to