fix: solve some inconsistencies between code and comments (#15394)

* Handle inconsistency

* Handle inconsistency

* Handle inconsistency

* Handle inconsistency

Author: YichiZhang0613

src/common/arrow/src/arrow/array/primitive/mutable.rs

@@ -320,7 +320,7 @@ impl<T: NativeType> MutablePrimitiveArray<T> {
     /// Note that if it is the first time a null appears in this array,
     /// this initializes the validity bitmap (`O(N)`).
     /// # Panic
-    /// Panics iff index is larger than `self.len()`.
+    /// Panics iff index is larger than or equal to `self.len()`.
     pub fn set(&mut self, index: usize, value: Option<T>) {
         assert!(index < self.len());
         // Safety:

src/common/arrow/src/arrow/array/union/mod.rs

@@ -89,7 +89,7 @@ impl UnionArray {
             .try_for_each(|(index, (data_type, child))| {
                 if data_type != child {
                     Err(Error::oos(format!(
-                        "The children DataTypes of a UnionArray must equal the children data types. 
+                        "The children DataTypes of a UnionArray must equal the children data types.
                          However, the field {index} has data type {data_type:?} but the value has data type {child:?}"
                     )))
                 } else {
@@ -244,7 +244,7 @@ impl UnionArray {
     /// # Implementation
     /// This operation is `O(F)` where `F` is the number of fields.
     /// # Panic
-    /// This function panics iff `offset + length >= self.len()`.
+    /// This function panics iff `offset + length > self.len()`.
     #[inline]
     pub fn slice(&mut self, offset: usize, length: usize) {
         assert!(

src/common/arrow/src/arrow/buffer/immutable.rs

@@ -162,7 +162,7 @@ impl<T> Buffer<T> {
 
     /// Slices this buffer starting at `offset`.
     /// # Panics
-    /// Panics iff `offset` is larger than `len`.
+    /// Panics iff `offset + length` is larger than `len`.
     #[inline]
     pub fn slice(&mut self, offset: usize, length: usize) {
         assert!(