You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@cassandra.apache.org by "Sylvain Lebresne (JIRA)" <ji...@apache.org> on 2011/09/26 18:13:27 UTC

[jira] [Updated] (CASSANDRA-3260) MergeIterator assertion on sources != empty can be thrown

     [ https://issues.apache.org/jira/browse/CASSANDRA-3260?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sylvain Lebresne updated CASSANDRA-3260:
----------------------------------------

    Attachment: 3260.patch

> MergeIterator assertion on sources != empty can be thrown
> ---------------------------------------------------------
>
>                 Key: CASSANDRA-3260
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3260
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 1.0.0
>            Reporter: Sylvain Lebresne
>            Assignee: Sylvain Lebresne
>            Priority: Trivial
>             Fix For: 1.0.0
>
>         Attachments: 3260.patch
>
>
> MergeIterator.get assert that it don't get an empty list of sources. This seems to at least not be the case in the unit test for some of tests (this don't make any test fail however, but there is a few stack trace thrown). I think it's pretty unnatural to "fail" on an empty list of sources and would force every caller to first take the empty case into account, so I propose to just remove that assertion.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira