Ian Maxon has posted comments on this change. Change subject: ASTERIXDB-1746: no excessive new byte[] allocation during a record build ......................................................................
Patch Set 2: Code-Review+2 I don't see anything wrong, but my only anxiety would be about thread safety. I don't see any real consideration of it elsewhere in ByteArrayAccessibleOutputStream though, so it's probably unfounded. -- To view, visit https://asterix-gerrit.ics.uci.edu/1376 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ibdbc6314abd72d2a93978da49ea406c3c38731d2 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Taewoo Kim <[email protected]> Gerrit-Reviewer: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-HasComments: No
