ctubbsii commented on PR #6463: URL: https://github.com/apache/accumulo/pull/6463#issuecomment-4928435403
This probably doesn't need to be done and the TODO can just be removed. Updating the public API creates churn that can be disruptive and more to maintain. I can see that it might have been worth doing this if somehow Mutation was altered to keep a reference to the ByteSequence and used that when it was serialized, instead of merely moving the toArray into a constructor, in order to possibly avoid an array copy, but it's not really necessary, and easier to just remove the TODO. -- 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]
