comment

apache · Jan 24, 2025 · 16c9e2f · 16c9e2f
1 parent b431cf0
commit 16c9e2f
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/vector/src/main/java/org/apache/arrow/vector/BaseVariableWidthViewVector.java b/vector/src/main/java/org/apache/arrow/vector/BaseVariableWidthViewVector.java
@@ -1368,8 +1368,8 @@ protected final void setBytes(int index, byte[] value, int start, int length) {
     int writePosition = index * ELEMENT_SIZE;
 
     if (length <= INLINE_SIZE) {
-      // to clear the memory segment of view being written to
-      // if it has been set
+      // Check if the memory segment has been written, and clear it if it has been set.
+      // It is recommended to batch initialize the viewBuffer before setBytes.
       if (viewBuffer.getLong(writePosition) != 0 || viewBuffer.getLong(writePosition + 8) != 0) {
         viewBuffer.setZero(writePosition, ELEMENT_SIZE);
       }
@@ -1414,8 +1414,8 @@ protected final void setBytes(int index, ArrowBuf valueBuf, int start, int lengt
     int writePosition = index * ELEMENT_SIZE;
 
     if (length <= INLINE_SIZE) {
-      // to clear the memory segment of view being written to
-      // if it has been set
+      // Check if the memory segment has been written, and clear it if it has been set.
+      // It is recommended to batch initialize the viewBuffer before setBytes.
       if (viewBuffer.getLong(writePosition) != 0 || viewBuffer.getLong(writePosition + 8) != 0) {
         viewBuffer.setZero(writePosition, ELEMENT_SIZE);
       }