accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "John Vines (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-633) FirstEntryInRowIterator is broken and has no test
Date Wed, 13 Jun 2012 22:08:42 GMT

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

John Vines commented on ACCUMULO-633:
-------------------------------------

So the middle row behavior can have some interesting edge cases. If we take the literal name,
then the attempting to return only the first Key is the proper thing to do. And I like that.
But you need to be careful there, as someone could seek to a specific Key, which could actually
be the first entry in the row. So this iterator will require seeking to the row and then checking
the top key to see if the Key specified is teh first entry before seeking to the next possible
row.
                
> FirstEntryInRowIterator is broken and has no test
> -------------------------------------------------
>
>                 Key: ACCUMULO-633
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-633
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.4.0, 1.3.5
>            Reporter: Billie Rinaldi
>            Assignee: Billie Rinaldi
>            Priority: Minor
>             Fix For: 1.5.0, 1.4.1
>
>
> In 1.4 and trunk, the iterator throws a NullPointerException when seeked.
> In 1.3 the iterator runs, but there is a question as to what it should do when it is
seeked to the middle of a row.  Currently, it returns the first key found within the range.
 I believe this should be changed to ignore the remaining portion of that row and return the
first key of the next row.  Should this change be made in 1.3, or should I leave it as is
and just change it in 1.4 and greater?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message