dlmarion commented on code in PR #4745:
URL: https://github.com/apache/accumulo/pull/4745#discussion_r1688505300
##########
core/src/main/java/org/apache/accumulo/core/data/ArrayByteSequence.java:
##########
@@ -180,4 +180,19 @@ public byte[] toArray() {
public String toString() {
return new String(data, offset, length, UTF_8);
}
+
+ /**
+ * Resets data, offset, and length of a backing array for this byte sequence
object.
+ *
+ * @since 3.1.0
+ */
+ public void reset(byte[] data, int offset, int length) {
Review Comment:
#4735 has been merged into `main`. The reset method now exists, but is
missing the bounds checking that you have added here.
--
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]
