Merge remote-tracking branch 'apache/master' into encode-skip-iter

.github/workflows/rust.yml

@@ -119,17 +119,20 @@ jobs:
         run: cargo update -p ahash --precise 0.8.7
       - name: Check arrow
         working-directory: arrow
-        run: cargo msrv verify
+        run: cargo msrv --log-target stdout verify
       - name: Check parquet
         working-directory: parquet
-        run: cargo msrv verify
+        run: cargo msrv --log-target stdout verify
       - name: Check arrow-flight
         working-directory: arrow-flight
-        run: cargo msrv verify
+        run: cargo msrv --log-target stdout verify
       - name: Downgrade object_store dependencies
         working-directory: object_store
-        # Necessary because 1.30.0 updates MSRV to 1.63
-        run: cargo update -p tokio --precise 1.29.1
+        # Necessary because tokio 1.30.0 updates MSRV to 1.63
+        # and url 2.5.1, updates to 1.67
+        run: |
+          cargo update -p tokio --precise 1.29.1
+          cargo update -p url --precise 2.5.0
       - name: Check object_store
         working-directory: object_store
-        run: cargo msrv verify
+        run: cargo msrv --log-target stdout verify

arrow-schema/src/schema.rs

@@ -331,6 +331,10 @@ impl Schema {
 
     /// Returns an immutable reference of a specific [`Field`] instance selected using an
     /// offset within the internal `fields` vector.
+    ///
+    /// # Panics
+    ///
+    /// Panics if index out of bounds
     pub fn field(&self, i: usize) -> &Field {
         &self.fields[i]
     }

parquet/src/arrow/array_reader/mod.rs

@@ -46,6 +46,7 @@ mod test_util;
 
 pub use builder::build_array_reader;
 pub use byte_array::make_byte_array_reader;
+#[allow(unused_imports)] // Only used for benchmarks
 pub use byte_array::make_byte_view_array_reader;
 pub use byte_array_dictionary::make_byte_array_dictionary_reader;
 #[allow(unused_imports)] // Only used for benchmarks