accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-1897) Bug in MemKeyConversionIterator constructor
Date Fri, 04 Apr 2014 02:03:17 GMT

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

ASF subversion and git services commented on ACCUMULO-1897:
-----------------------------------------------------------

Commit 75d62a46279aa448c386d7408a4268b0fe842c20 in accumulo's branch refs/heads/master from
[~busbey]
[ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=75d62a4 ]

ACCUMULO-1897 removes unneeded constructor for MemKeyConversionIterator.

Signed-off-by: Mike Drob <mdrob@cloudera.com>


> Bug in MemKeyConversionIterator constructor
> -------------------------------------------
>
>                 Key: ACCUMULO-1897
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1897
>             Project: Accumulo
>          Issue Type: Bug
>          Components: tserver
>    Affects Versions: 1.5.0, 1.5.1
>            Reporter: Bill Havanki
>            Assignee: Sean Busbey
>             Fix For: 1.5.2, 1.6.0
>
>         Attachments: ACCUMULO-1897.1.patch.txt
>
>
> The constructor for {{InMemoryMap.MemKeyConversionIterator}} which takes a starting key
does not correctly clone the key. The parameter to the constructor is {{startKey}}, but the
code looks at the {{currKey}} field to check whether a key is available.
> {code:java}
> if (currKey != null)  // <- should be startKey != null
>         currKey = (MemKey) startKey.clone();
> {code}
> This class was introduced in version 1.5.0, and is not present in 1.4.x.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message