Chore: Use full item paths for every import is vortex-vector (#5509)

Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>

Author: connortsui20

vortex-vector/src/binaryview/mod.rs

@@ -15,7 +15,8 @@ pub use vector_mut::*;
 pub use view::*;
 use vortex_error::vortex_panic;
 
-use crate::{Vector, VectorMut};
+use crate::Vector;
+use crate::VectorMut;
 
 mod scalar;
 mod types;

vortex-vector/src/binaryview/scalar.rs

@@ -1,10 +1,14 @@
 // SPDX-License-Identifier: Apache-2.0
 // SPDX-FileCopyrightText: Copyright the Vortex contributors
 
-use crate::binaryview::{
-    BinaryType, BinaryViewType, BinaryViewTypeUpcast, BinaryViewVectorMut, StringType,
-};
-use crate::{Scalar, ScalarOps, VectorMutOps};
+use crate::Scalar;
+use crate::ScalarOps;
+use crate::VectorMutOps;
+use crate::binaryview::BinaryType;
+use crate::binaryview::BinaryViewType;
+use crate::binaryview::BinaryViewTypeUpcast;
+use crate::binaryview::BinaryViewVectorMut;
+use crate::binaryview::StringType;
 
 /// A scalar value for types that implement [`BinaryViewType`].
 #[derive(Debug)]

vortex-vector/src/binaryview/types.rs

@@ -5,10 +5,13 @@
 
 use std::fmt::Debug;
 
-use vortex_buffer::{BufferString, ByteBuffer};
+use vortex_buffer::BufferString;
+use vortex_buffer::ByteBuffer;
 
-use crate::binaryview::{BinaryViewVector, BinaryViewVectorMut};
-use crate::{Vector, VectorMut};
+use crate::Vector;
+use crate::VectorMut;
+use crate::binaryview::BinaryViewVector;
+use crate::binaryview::BinaryViewVectorMut;
 
 impl<T: BinaryViewType> From<BinaryViewVector<T>> for Vector {
     fn from(value: BinaryViewVector<T>) -> Self {

vortex-vector/src/binaryview/vector.rs

@@ -7,14 +7,20 @@ use std::fmt::Debug;
 use std::ops::RangeBounds;
 use std::sync::Arc;
 
-use vortex_buffer::{Alignment, Buffer, ByteBuffer};
-use vortex_error::{VortexExpect, VortexResult, vortex_ensure};
+use vortex_buffer::Alignment;
+use vortex_buffer::Buffer;
+use vortex_buffer::ByteBuffer;
+use vortex_error::VortexExpect;
+use vortex_error::VortexResult;
+use vortex_error::vortex_ensure;
 use vortex_mask::Mask;
 
 use crate::VectorOps;
+use crate::binaryview::BinaryViewScalar;
+use crate::binaryview::BinaryViewType;
 use crate::binaryview::vector_mut::BinaryViewVectorMut;
-use crate::binaryview::view::{BinaryView, validate_views};
-use crate::binaryview::{BinaryViewScalar, BinaryViewType};
+use crate::binaryview::view::BinaryView;
+use crate::binaryview::view::validate_views;
 
 /// A variable-length binary vector.
 ///
@@ -282,12 +288,15 @@ impl<T: BinaryViewType> VectorOps for BinaryViewVector<T> {
 mod tests {
     use std::sync::Arc;
 
-    use vortex_buffer::{ByteBuffer, buffer};
+    use vortex_buffer::ByteBuffer;
+    use vortex_buffer::buffer;
     use vortex_mask::Mask;
 
+    use crate::VectorMutOps;
+    use crate::VectorOps;
+    use crate::binaryview::StringVector;
+    use crate::binaryview::StringVectorMut;
     use crate::binaryview::view::BinaryView;
-    use crate::binaryview::{StringVector, StringVectorMut};
-    use crate::{VectorMutOps, VectorOps};
 
     #[test]
     #[should_panic(expected = "views buffer length 1 != validity length 100")]

vortex-vector/src/binaryview/vector_mut.rs

@@ -5,14 +5,21 @@
 
 use std::sync::Arc;
 
-use vortex_buffer::{BufferMut, ByteBuffer, ByteBufferMut};
-use vortex_error::{VortexExpect, VortexResult, vortex_ensure};
+use vortex_buffer::BufferMut;
+use vortex_buffer::ByteBuffer;
+use vortex_buffer::ByteBufferMut;
+use vortex_error::VortexExpect;
+use vortex_error::VortexResult;
+use vortex_error::vortex_ensure;
 use vortex_mask::MaskMut;
 
+use crate::VectorMutOps;
+use crate::VectorOps;
+use crate::binaryview::BinaryViewScalar;
+use crate::binaryview::BinaryViewType;
 use crate::binaryview::vector::BinaryViewVector;
-use crate::binaryview::view::{BinaryView, validate_views};
-use crate::binaryview::{BinaryViewScalar, BinaryViewType};
-use crate::{VectorMutOps, VectorOps};
+use crate::binaryview::view::BinaryView;
+use crate::binaryview::view::validate_views;
 
 // Default capacity for new string data buffers of 2MiB.
 const BUFFER_CAPACITY: usize = 2 * 1024 * 1024;
@@ -310,12 +317,17 @@ mod tests {
     use std::ops::Deref;
     use std::sync::Arc;
 
-    use vortex_buffer::{ByteBuffer, buffer, buffer_mut};
-    use vortex_mask::{Mask, MaskMut};
+    use vortex_buffer::ByteBuffer;
+    use vortex_buffer::buffer;
+    use vortex_buffer::buffer_mut;
+    use vortex_mask::Mask;
+    use vortex_mask::MaskMut;
 
+    use crate::VectorMutOps;
+    use crate::VectorOps;
+    use crate::binaryview::StringVector;
+    use crate::binaryview::StringVectorMut;
     use crate::binaryview::view::BinaryView;
-    use crate::binaryview::{StringVector, StringVectorMut};
-    use crate::{VectorMutOps, VectorOps};
 
     #[test]
     fn test_basic() {

vortex-vector/src/binaryview/view.rs

@@ -4,12 +4,17 @@
 //! The 16-byte view struct stored in variable-length binary vectors.
 
 use std::fmt;
-use std::hash::{Hash, Hasher};
+use std::hash::Hash;
+use std::hash::Hasher;
 use std::ops::Range;
 
-use static_assertions::{assert_eq_align, assert_eq_size};
+use static_assertions::assert_eq_align;
+use static_assertions::assert_eq_size;
 use vortex_buffer::ByteBuffer;
-use vortex_error::{VortexResult, VortexUnwrap, vortex_ensure, vortex_err};
+use vortex_error::VortexResult;
+use vortex_error::VortexUnwrap;
+use vortex_error::vortex_ensure;
+use vortex_error::vortex_err;
 
 /// A view over a variable-length binary value.
 ///

vortex-vector/src/bool/mod.rs

@@ -65,7 +65,8 @@ pub use scalar::BoolScalar;
 
 mod iter;
 
-use crate::{Vector, VectorMut};
+use crate::Vector;
+use crate::VectorMut;
 
 impl From<BoolVector> for Vector {
     fn from(v: BoolVector) -> Self {

vortex-vector/src/bool/scalar.rs

@@ -1,8 +1,11 @@
 // SPDX-License-Identifier: Apache-2.0
 // SPDX-FileCopyrightText: Copyright the Vortex contributors
 
+use crate::Scalar;
+use crate::ScalarOps;
+use crate::VectorMut;
+use crate::VectorMutOps;
 use crate::bool::BoolVectorMut;
-use crate::{Scalar, ScalarOps, VectorMut, VectorMutOps};
 
 /// A scalar value for boolean types.
 #[derive(Debug)]

vortex-vector/src/bool/vector.rs

@@ -7,11 +7,14 @@ use std::fmt::Debug;
 use std::ops::RangeBounds;
 
 use vortex_buffer::BitBuffer;
-use vortex_error::{VortexExpect, VortexResult, vortex_ensure};
+use vortex_error::VortexExpect;
+use vortex_error::VortexResult;
+use vortex_error::vortex_ensure;
 use vortex_mask::Mask;
 
 use crate::VectorOps;
-use crate::bool::{BoolScalar, BoolVectorMut};
+use crate::bool::BoolScalar;
+use crate::bool::BoolVectorMut;
 
 /// An immutable vector of boolean values.
 ///

vortex-vector/src/bool/vector_mut.rs

@@ -4,11 +4,15 @@
 //! Definition and implementation of [`BoolVectorMut`].
 
 use vortex_buffer::BitBufferMut;
-use vortex_error::{VortexExpect, VortexResult, vortex_ensure};
+use vortex_error::VortexExpect;
+use vortex_error::VortexResult;
+use vortex_error::vortex_ensure;
 use vortex_mask::MaskMut;
 
-use crate::bool::{BoolScalar, BoolVector};
-use crate::{VectorMutOps, VectorOps};
+use crate::VectorMutOps;
+use crate::VectorOps;
+use crate::bool::BoolScalar;
+use crate::bool::BoolVector;
 
 /// A mutable vector of boolean values.
 ///