accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Wall (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-4377) Document and add test for Key constructors copy behavior
Date Tue, 02 Aug 2016 23:02:20 GMT

    [ 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)

Mime
View raw message