[ 
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)

Reply via email to