keith-turner commented on code in PR #4745:
URL: https://github.com/apache/accumulo/pull/4745#discussion_r1687021630
##########
core/src/main/java/org/apache/accumulo/core/data/ArrayByteSequence.java:
##########
@@ -180,4 +180,21 @@ public byte[] toArray() {
public String toString() {
return new String(data, offset, length, UTF_8);
}
+
+ public void setArray(byte[] data, int offset, int length) {
Review Comment:
Feel free to ignore this comment as its personal preference. I like the
method name `reset` used in #4735, so could use that instead of `setArray`.
This is a new public API method that is being introduced, so can call it
whatever we like. Do not need to use the same method name as
MutableByteSequence
--
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]
