ctubbsii commented on code in PR #4779:
URL: https://github.com/apache/accumulo/pull/4779#discussion_r1708094122
##########
core/src/main/java/org/apache/accumulo/core/data/Key.java:
##########
@@ -657,6 +657,21 @@ public ByteSequence getRowData() {
return new ArrayByteSequence(row);
}
+ /**
+ * Writes the row ID into the given <code>ArrayByteSequence</code> without
allocating new object
+ * by using {@link
org.apache.accumulo.core.data.ArrayByteSequence#reset(byte[], int, int)}
+ * method. Using this updates existing ArrayByteSequence object with
reference to row data, rather
+ * than copying row data.
+ *
+ * @param r <code>ArrayByteSequence</code> object to copy into
Review Comment:
I'm not sure why these javadocs use `<code>`, but it makes more sense to be
using the javadoc tag for `{@link ArrayByteSequence}` instead, I think.
--
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]
