fmt

Author: alamb

parquet-variant-compute/src/variant_array.rs

@@ -150,7 +150,8 @@ impl VariantArray {
         // This would be a lot simpler if ShreddingState were just a pair of Option... we already
         // have everything we need.
         let inner = builder.build();
-        let shredding_state = ShreddingState::try_new(metadata.clone(), value, typed_value).unwrap(); // valid by construction
+        let shredding_state =
+            ShreddingState::try_new(metadata.clone(), value, typed_value).unwrap(); // valid by construction
         Self {
             inner,
             metadata,
@@ -207,7 +208,9 @@ impl VariantArray {
                     typed_value_to_variant(typed_value, index)
                 }
             }
-            ShreddingState::PartiallyShredded { value, typed_value, .. } => {
+            ShreddingState::PartiallyShredded {
+                value, typed_value, ..
+            } => {
                 // PartiallyShredded case (formerly ImperfectlyShredded)
                 if typed_value.is_null(index) {
                     Variant::new(self.metadata.value(index), value.value(index))
@@ -313,9 +316,11 @@ impl ShreddedVariantFieldArray {
         };
 
         // Extract value and typed_value fields (metadata is not expected in ShreddedVariantFieldArray)
-        let value = inner_struct.column_by_name("value").and_then(|col| col.as_binary_view_opt().cloned());
+        let value = inner_struct
+            .column_by_name("value")
+            .and_then(|col| col.as_binary_view_opt().cloned());
         let typed_value = inner_struct.column_by_name("typed_value").cloned();
-        
+
         // Use a dummy metadata for the constructor (ShreddedVariantFieldArray doesn't have metadata)
         let dummy_metadata = arrow::array::BinaryViewArray::new_null(inner_struct.len());
 
@@ -387,8 +392,8 @@ impl Array for ShreddedVariantFieldArray {
     }
 
     fn nulls(&self) -> Option<&NullBuffer> {
-        // According to the shredding spec, ShreddedVariantFieldArray should be 
-        // physically non-nullable - SQL NULL is inferred by both value and 
+        // According to the shredding spec, ShreddedVariantFieldArray should be
+        // physically non-nullable - SQL NULL is inferred by both value and
         // typed_value being physically NULL
         None
     }
@@ -423,13 +428,13 @@ impl Array for ShreddedVariantFieldArray {
 #[derive(Debug)]
 pub enum ShreddingState {
     /// This variant has no typed_value field
-    Unshredded { 
+    Unshredded {
         metadata: BinaryViewArray,
         value: BinaryViewArray,
     },
     /// This variant has a typed_value field and no value field
     /// meaning it is the shredded type
-    Typed { 
+    Typed {
         metadata: BinaryViewArray,
         typed_value: ArrayRef,
     },
@@ -454,9 +459,7 @@ pub enum ShreddingState {
     /// Note: By strict spec interpretation, this should only be valid for shredded object fields,
     /// not top-level variants. However, we allow it and treat as Variant::Null for pragmatic
     /// handling of missing data.
-    AllNull { 
-        metadata: BinaryViewArray,
-    },
+    AllNull { metadata: BinaryViewArray },
 }
 
 impl ShreddingState {