[CASSANDRA-3260] MergeIterator assertion on sources != empty can be thrown

• Type: Bug
• Status: Resolved
• Priority: Low
• Resolution: Fixed
• Fix Version/s: 1.0.0
• Severity: Low
• Assignee: Sylvain Lebresne
• Reporter: Sylvain Lebresne

Authors

Sylvain Lebresne

Reviewers

Jonathan Ellis

• Created: 26/Sep/11 16:11
• Updated: 16/Apr/19 09:32
• Resolved: 27/Sep/11 13:16

Description

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.