You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "Mike Drob (JIRA)" <ji...@apache.org> on 2014/04/01 01:37:28 UTC
[jira] [Commented] (ACCUMULO-2600) Key.clone() should return type
Key
[ https://issues.apache.org/jira/browse/ACCUMULO-2600?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13955882#comment-13955882 ]
Mike Drob commented on ACCUMULO-2600:
-------------------------------------
Also, the same logic can be applied to {{MemKey}}. Which, incidentally, might be broken too.
{code}
public MemKeyConversionIterator(SortedKeyValueIterator<Key,Value> source, MemKey startKey) {
this(source);
try {
if (currKey != null)
currKey = (MemKey) startKey.clone();
} catch (CloneNotSupportedException e) {
// MemKey is supported
}
}
{code}
This is in the constructor, and I think that {{currKey}} will always be null.
> Key.clone() should return type Key
> ----------------------------------
>
> Key: ACCUMULO-2600
> URL: https://issues.apache.org/jira/browse/ACCUMULO-2600
> Project: Accumulo
> Issue Type: Bug
> Components: client
> Affects Versions: 1.4.0, 1.5.0
> Reporter: Mike Drob
> Assignee: Mike Drob
> Priority: Critical
> Labels: API
> Fix For: 1.4.6, 1.5.2, 1.6.0
>
>
> {{Key.clone()}} can return {{Key}} instead of {{Object}} like it does currently. This is nicer on end-users of the API.
--
This message was sent by Atlassian JIRA
(v6.2#6252)