jduo commented on code in PR #38266:
URL: https://github.com/apache/arrow/pull/38266#discussion_r1361243600
##########
java/vector/src/main/java/org/apache/arrow/vector/FixedSizeBinaryVector.java:
##########
@@ -116,6 +117,18 @@ public byte[] get(int index) {
return dst;
}
+ /**
+ * Read the value at the given position to the given output buffer.
+ * The caller is responsible for checking for nullity first.
+ *
+ * @param index position of element.
+ * @param outputBuffer the buffer to write into.
+ */
+ public void readBytes(int index, ReusableBuffer<?> outputBuffer) {
Review Comment:
I'll go with read(). Avoiding using getObject() since it sounds too much
like it should return a value vs. populate a buffer.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
