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

Branimir Lambov commented on CASSANDRA-18756:
---------------------------------------------

[~mck], could you do the second review this small patch which corrects a 
problem with CASSANDRA-13418? Your name came up as a reviewer for that ticket 
and it would be great to get the opinion of someone who has some context on it.

> TimeWindowCompactionStrategy with unsafe_aggressive_sstable_expiration keeps 
> overlaping SSTable references
> ----------------------------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-18756
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-18756
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Local/Compaction
>            Reporter: Branimir Lambov
>            Priority: Normal
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> When {{unsafe_aggressive_sstable_expiration}} is turned on, TWCS should not 
> create or maintain an iterator of overlapping sstables. However, because 
> {{TimeWindowCompactionController}} inherits from {{CompactionController}} and 
> only sets {{ignoreOverlaps}} after the base class has constructed the overlap 
> iterator, it ends up making an overlap iterator and then never updating it.
> The end result is that such a compaction keeps references to lots of and 
> likely _all_ other SSTables on the node and thus delays the deletion of 
> obsolete ones by hours or even days.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to