cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Paulo Motta (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-14081) Remove AbstractCompactionStrategy.replaceFlushed
Date Fri, 01 Dec 2017 13:47:00 GMT


Paulo Motta commented on CASSANDRA-14081:

bq.  should we remove ACS#getMemtableReservedSize() and ACS#isAffectedByMeteredFlusher() at
the same time?

Agreed. In the same spirit I removed the deprecated methods {{enable}}, {{disabled}} and {{shouldBeEnabled}}
from {{AbstractCompactionStrategy}}, do you think this is OK?

Final patch [here|]. CI looks
good (just some [unrelated dtest failures|])

> Remove AbstractCompactionStrategy.replaceFlushed
> ------------------------------------------------
>                 Key: CASSANDRA-14081
>                 URL:
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Paulo Motta
>            Assignee: Paulo Motta
>            Priority: Minor
>         Attachments: dtest14081.png
> I didn't find a reason for why we need to send flush notifications from CFs -> CSM
-> Tracker, if we can bypass the CSM and send directly to the tracker from the CFS (and
handle it on the CSM via {{SSTableAddedNotification}}.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message