accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Keith Turner (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-794) FirstEntryInRowIterator can scan through entire last row in range
Date Wed, 10 Oct 2012 15:33:02 GMT

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

Keith Turner commented on ACCUMULO-794:
---------------------------------------

bq. I was thinking that would be non-trivial since this is a performance issue and not a correctness
issue. However, I suppose one could write a source iterator that complains if next is called
too many times in one row before seek is called. Still seems like more trouble than it's worth.

I agree.  I did not realize it was just a performance issue.  I was not paying close attention.
 I took a quick look at the bug and thought the iterator was returning more data than it should.
 That do being said, I do like Billie's suggestion.  More test never hurt. Well, as long as
they complete in a reasonable amount of time.


                
> FirstEntryInRowIterator can scan through entire last row in range
> -----------------------------------------------------------------
>
>                 Key: ACCUMULO-794
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-794
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.4.1
>            Reporter: Adam Fuchs
>            Assignee: Adam Fuchs
>             Fix For: 1.5.0, 1.4.2
>
>
> If the range passed to the FirstEntryInRowIterator is seeked with a Range that has an
end key, the FirstEntryInRowIterator will scan through all of the entries in the last row
that match the range. This is due to a logic bug -- the if statement on line 93 needs an else
clause.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message