mzabaluev commented on code in PR #9534:
URL: https://github.com/apache/arrow-rs/pull/9534#discussion_r2918059676


##########
arrow-avro/src/reader/async_reader/mod.rs:
##########
@@ -1538,6 +1571,87 @@ mod tests {
         assert!(err.to_string().contains("Duplicate projection index"));
     }
 
+    #[tokio::test]
+    async fn test_arrow_schema_from_reader_no_reader_schema() {
+        // Use a very small header size hint to force multiple fetches
+        let file = arrow_test_data("avro/alltypes_plain.avro");
+        let store: Arc<dyn ObjectStore> = Arc::new(LocalFileSystem::new());
+        let location = Path::from_filesystem_path(&file).unwrap();
+        let file_size = store.head(&location).await.unwrap().size;
+
+        let file_reader = AvroObjectReader::new(store, location);

Review Comment:
   I have added clarifying comments explaining the purpose and differences of 
code in each of the added cases. Hope this helps.



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