remove as_fixed_size_array which is only useful for decimal

jorgecarleitao · Jan 3, 2022 · b67e9b4 · b67e9b4
1 parent 55d01ec
commit b67e9b4
Showing 1 changed file with 10 additions and 11 deletions.
diff --git a/src/array/fixed_size_binary/mutable.rs b/src/array/fixed_size_binary/mutable.rs
@@ -162,15 +162,6 @@ impl MutableFixedSizeBinaryArray {
             validity.shrink_to_fit()
         }
     }
-
-    /// Creates an owned [`FixedSizeBinaryArray`] from current data
-    pub fn as_fixed_size_array(&mut self) -> FixedSizeBinaryArray {
-        FixedSizeBinaryArray::from_data(
-            DataType::FixedSizeBinary(self.size),
-            std::mem::take(&mut self.values).into(),
-            std::mem::take(&mut self.validity).map(|x| x.into()),
-        )
-    }
 }
 
 /// Accessors
@@ -196,11 +187,19 @@ impl MutableArray for MutableFixedSizeBinaryArray {
     }
 
     fn as_box(&mut self) -> Box<dyn Array> {
-        Box::new(self.as_fixed_size_array())
+        Box::new(FixedSizeBinaryArray::from_data(
+            DataType::FixedSizeBinary(self.size),
+            std::mem::take(&mut self.values).into(),
+            std::mem::take(&mut self.validity).map(|x| x.into()),
+        ))
     }
 
     fn as_arc(&mut self) -> Arc<dyn Array> {
-        Arc::new(self.as_fixed_size_array())
+        Arc::new(FixedSizeBinaryArray::from_data(
+            DataType::FixedSizeBinary(self.size),
+            std::mem::take(&mut self.values).into(),
+            std::mem::take(&mut self.validity).map(|x| x.into()),
+        ))
     }
 
     fn data_type(&self) -> &DataType {