Apply suggestions from code review

Co-authored-by: Jeffrey Vo <[email protected]>
apache · Jan 5, 2025 · 1d48865 · 1d48865
1 parent e8a796a
commit 1d48865
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/arrow-buffer/src/buffer/immutable.rs b/arrow-buffer/src/buffer/immutable.rs
@@ -236,7 +236,7 @@ impl Buffer {
 
     /// Returns a new [Buffer] that is a slice of this buffer starting at `offset`.
     ///
-    /// This function is `O(1)` and does not copy any data, and allows the
+    /// This function is `O(1)` and does not copy any data, allowing the
     /// same memory region to be shared between buffers.
     ///
     /// # Panics
@@ -272,7 +272,7 @@ impl Buffer {
     /// Returns a new [Buffer] that is a slice of this buffer starting at `offset`,
     /// with `length` bytes.
     ///
-    /// This function is `O(1)` and does not copy any data and  allows the same
+    /// This function is `O(1)` and does not copy any data, allowing the same
     /// memory region to be shared between buffers.
     ///
     /// # Panics
@@ -454,7 +454,7 @@ impl<T: ArrowNativeType> From<ScalarBuffer<T>> for Buffer {
     }
 }
 
-/// Convert from internal `Bytes`, not [`bytes::Bytes`] to `Buffer`
+/// Convert from internal `Bytes` (not [`bytes::Bytes`]) to `Buffer`
 impl From<Bytes> for Buffer {
     #[inline]
     fn from(bytes: Bytes) -> Self {

diff --git a/parquet/src/arrow/array_reader/byte_view_array.rs b/parquet/src/arrow/array_reader/byte_view_array.rs
@@ -316,7 +316,7 @@ impl ByteViewArrayDecoderPlain {
     }
 
     pub fn read(&mut self, output: &mut ViewBuffer, len: usize) -> Result<usize> {
-        // Zero copy convert `bytes::Bytes` into `arrow_buffer:Buffer`
+        // Zero copy convert `bytes::Bytes` into `arrow_buffer::Buffer`
         let buf = arrow_buffer::Buffer::from(self.buf.clone());
         let block_id = output.append_block(buf);
 
@@ -548,7 +548,7 @@ impl ByteViewArrayDecoderDeltaLength {
 
         let src_lengths = &self.lengths[self.length_offset..self.length_offset + to_read];
 
-        // Zero copy convert `bytes::Bytes` into `arrow_buffer:Buffer`
+        // Zero copy convert `bytes::Bytes` into `arrow_buffer::Buffer`
         let bytes = Buffer::from(self.data.clone());
         let block_id = output.append_block(bytes);