[
https://issues.apache.org/jira/browse/CASSANDRA-3260?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13114775#comment-13114775
]
Jonathan Ellis commented on CASSANDRA-3260:
-------------------------------------------
Should also then change the next line from
if (sources.size() == 1)
to
if (sources.size() <= 1)
+1 w/ that
> 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
