Minor: try and avoid an allocation creating GenericByteViewArray from ArrayData (apache#9156)

# Which issue does this PR close?

- part of apache#9061
- follow on apache#9114


# Rationale for this change

@scovich noted in
apache#9114 (comment) that
calling `Vec::remove` does an extra copy and that `Vec::from` doesn't
actually reuse the allocation the way I thought it did


# What changes are included in this PR?

Build the Arc for buffers directly

# Are these changes tested?

BY existing tests

# Are there any user-facing changes?

<!--
If there are user-facing changes then we may require documentation to be
updated before approving the PR.

If there are any breaking changes to public APIs, please call them out.
-->

Author: alamb

arrow-array/src/array/byte_view_array.rs

@@ -988,14 +988,16 @@ impl<'a, T: ByteViewType + ?Sized> IntoIterator for &'a GenericByteViewArray<T>
 
 impl<T: ByteViewType + ?Sized> From<ArrayData> for GenericByteViewArray<T> {
     fn from(data: ArrayData) -> Self {
-        let (_data_type, len, nulls, offset, mut buffers, _child_data) = data.into_parts();
-        let views = buffers.remove(0); // need to maintain order of remaining buffers
-        let buffers = Arc::from(buffers);
-        let views = ScalarBuffer::new(views, offset, len);
+        let (_data_type, len, nulls, offset, buffers, _child_data) = data.into_parts();
+
+        let mut buffers = buffers.into_iter();
+        // first buffer is views, remaining are data buffers
+        let views = ScalarBuffer::new(buffers.next().unwrap(), offset, len);
+
         Self {
             data_type: T::DATA_TYPE,
             views,
-            buffers,
+            buffers: Arc::from_iter(buffers),
             nulls,
             phantom: Default::default(),
         }