dlmarion commented on code in PR #4735:
URL: https://github.com/apache/accumulo/pull/4735#discussion_r1672765085
##########
core/src/main/java/org/apache/accumulo/core/data/ArrayByteSequence.java:
##########
@@ -154,6 +154,15 @@ public int offset() {
return offset;
}
+ /**
+ * Reset the backing array for this byte sequence object. This is useful for
object re-use.
+ */
+ public void reset(byte[] data) {
Review Comment:
> did not look at what uses MutableByteSequence
It's only used in RelativeKey and LocalityGroupUtil. I'm thinking
MutableByteSequence could be removed and the functionality collapsed into
ArrayByteSequence. That might be a different PR though. I can create an issue
for it.
--
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]
