accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sean Busbey (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-3168) WholeRowIterator.decode could return NavigableMap instead of SortedMap
Date Thu, 25 Sep 2014 18:32:35 GMT

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

Sean Busbey commented on ACCUMULO-3168:
---------------------------------------

We should not return an actual implementation like TreeMap from a method that is going to
get used by end users. We should return the least-specific Interface that will give users
the things we want to promise to keep giving them.

I'm +0 on NavigableMap, +1 on leaving it as SortedMap, and -1 on anything more or less specific.

> WholeRowIterator.decode could return NavigableMap instead of SortedMap
> ----------------------------------------------------------------------
>
>                 Key: ACCUMULO-3168
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3168
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: client
>            Reporter: Mike Drob
>            Priority: Minor
>              Labels: newbie
>             Fix For: 1.7.0
>
>
> {{WRI.decode()}} returns a TreeMap declared as a SortedMap. We could give users access
to useful methods on NavigableMap without limiting which implementation we can use out of
the Java collections packages, if we decide to swap it out at a later date.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message