[jira] [Commented] (ACCUMULO-4377) Document and add test for Key constructors copy behavior

["Michael Wall (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ACCUMULO-4377?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15404967#comment-15404967 ] 

Michael Wall commented on ACCUMULO-4377:
----------------------------------------

[~kturner] can I move this to 1.8.1?

> Document and add test for Key constructors copy behavior
> --------------------------------------------------------
>
>                 Key: ACCUMULO-4377
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4377
>             Project: Accumulo
>          Issue Type: Improvement
>            Reporter: Keith Turner
>             Fix For: 1.8.0
>
>
> While looking at [Github PR #125|https://github.com/apache/accumulo/pull/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 message was sent by Atlassian JIRA
(v6.3.4#6332)