removes redundant methods

arrow/src/array/array.rs

@@ -60,7 +60,7 @@ pub trait Array: fmt::Debug + Send + Sync + JsonEqual {
     /// Returns a reference to the underlying data of this array.
     fn data(&self) -> &ArrayData;
 
-    /// Returns underlying data of this array.
+    /// Returns the underlying data of this array.
     fn into_data(self) -> ArrayData;
 
     /// Returns a reference-counted pointer to the underlying data of this array.

arrow/src/array/array_primitive.rs

@@ -166,11 +166,6 @@ impl<T: ArrowPrimitiveType> PrimitiveArray<T> {
     ) -> impl Iterator<Item = Option<T::Native>> + 'a {
         indexes.map(|opt_index| opt_index.map(|index| self.value_unchecked(index)))
     }
-
-    /// Returns the backing [`ArrayData`] of this [`PrimitiveArray`]
-    pub fn into_data(self) -> ArrayData {
-        self.into()
-    }
 }
 
 impl<T: ArrowPrimitiveType> From<PrimitiveArray<T>> for ArrayData {

arrow/src/array/array_string.rs

@@ -195,11 +195,6 @@ impl<OffsetSize: OffsetSizeTrait> GenericStringArray<OffsetSize> {
     ) -> impl Iterator<Item = Option<&str>> + 'a {
         indexes.map(|opt_index| opt_index.map(|index| self.value_unchecked(index)))
     }
-
-    /// Returns the backing [`ArrayData`] of this [`GenericStringArray`]
-    pub fn into_data(self) -> ArrayData {
-        self.into()
-    }
 }
 
 impl<'a, Ptr, OffsetSize: OffsetSizeTrait> FromIterator<&'a Option<Ptr>>

arrow/src/array/builder/primitive_dictionary_builder.rs

@@ -19,7 +19,7 @@ use std::any::Any;
 use std::collections::HashMap;
 use std::sync::Arc;
 
-use crate::array::{ArrayRef, ArrowPrimitiveType, DictionaryArray};
+use crate::array::{Array, ArrayRef, ArrowPrimitiveType, DictionaryArray};
 use crate::datatypes::{ArrowNativeType, DataType, ToByteSlice};
 use crate::error::{ArrowError, Result};