You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2019/04/17 23:35:18 UTC

[GitHub] [accumulo] DonResnik opened a new issue #1104: Document and add test for Key constructors copy behavior

DonResnik opened a new issue #1104: Document and add test for Key constructors copy behavior
URL: https://github.com/apache/accumulo/issues/1104
 
 
   While looking at Github PR #125 I thought it was nice that the new constructors documented the copy behavior. It would be nice to update the javadoc for the constructors that existed before the PR to mention the copy behavior.
   Also all of the constructors copy behavior should be tested in KeyTest.java. Should do things like pass in byte array to constructor, modify bye array, verify Key does not reflect changes.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services