From 49ada2a42edb69a5d884c19d775742bab4a9ea7c Mon Sep 17 00:00:00 2001
From: Deepak Cherian <deepak@cherian.net>
Date: Mon, 12 Aug 2024 10:28:44 -0600
Subject: [PATCH] one more cleanup

---
 src/structure.rs | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/src/structure.rs b/src/structure.rs
index e73d0f6b..879bbf6f 100644
--- a/src/structure.rs
+++ b/src/structure.rs
@@ -102,8 +102,7 @@ impl StructureTable {
         };
 
         let encoded_fill_value = self.batch.column_by_name("fill_value")?
-                .as_any()
-                .downcast_ref::<BinaryArray>()
+                .as_binary_opt::<i32>()
                 .map(|x| x.value(idx))
         ;
         let fill_value = encoded_fill_value
@@ -567,11 +566,7 @@ pub fn mk_structure_table<T: IntoIterator<Item = NodeStructure>>(
             true,
         ),
         Field::new("chunk_key_encoding", arrow::datatypes::DataType::UInt8, true),
-        Field::new(
-            "fill_value",
-            arrow::datatypes::DataType::Binary,
-            true,
-        ),
+        Field::new("fill_value", arrow::datatypes::DataType::Binary, true),
         Field::new("codecs", arrow::datatypes::DataType::Utf8, true),
         Field::new("storage_transformers", arrow::datatypes::DataType::Utf8, true),
         Field::new_list(