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

Jordan West commented on CASSANDRA-15445:
-----------------------------------------

+1. LGTM. The unit tests failures look unrelated and likely flaky. It could be 
worth running the jdk11 tests as well. One minor suggestion, totally optional: 
for future readers, it might be nice to have some comments in 
{{testDisableSTCSInL0}}. E.g. "setup some sstables in L1 that should be the 
first compaction candidates after STCS in L0 is disabled" and "create a bunch 
of sstables in L0 so it looks behind". 

 

 

> Make it possible to disable STCS-in-L0 during runtime
> -----------------------------------------------------
>
>                 Key: CASSANDRA-15445
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-15445
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Local/Compaction
>            Reporter: Marcus Eriksson
>            Assignee: Marcus Eriksson
>            Priority: Normal
>             Fix For: 4.x
>
>
> It should be possible for operators to disable STCS-in-L0 without bouncing 
> the node.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to