cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Marcus Eriksson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CASSANDRA-11660) Dubious call to remove in CompactionManager.java
Date Tue, 26 Apr 2016 11:26:12 GMT

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

Marcus Eriksson commented on CASSANDRA-11660:
---------------------------------------------

Did you see the exception or did you find it by code inspection?

We should probably remove that entire block of code, iirc it was a workaround before we had
proper life cycle handling of sstables

> Dubious call to remove in CompactionManager.java
> ------------------------------------------------
>
>                 Key: CASSANDRA-11660
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-11660
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Compaction
>            Reporter: Max Schaefer
>
> I'm surprised by [this|https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/db/compaction/CompactionManager.java#L1382]
call to {{Iterator.remove()}} in {{org.apache.cassandra.db.compaction.antiCompactGroup}}:
the iterator in question seems to come from [org.apache.cassandra.db.lifecycle.LifecycleTransaction.originals()|https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/db/lifecycle/LifecycleTransaction.java#L419],
which returns an unmodifiable set, so I would expect this call to always fail with an {{UnsupportedOperationException}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message