[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2016-01-24 Thread Stefania (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15114791#comment-15114791 ] Stefania commented on CASSANDRA-9949: - CI looks good, ready for review [~krummas]. >

[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2016-01-22 Thread Stefania (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15112131#comment-15112131 ] Stefania commented on CASSANDRA-9949: - The 3.0 patch is ready as well. This patch is much simpler

[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2016-01-19 Thread Marcus Eriksson (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15106541#comment-15106541 ] Marcus Eriksson commented on CASSANDRA-9949: I doubt it matters much which timestamp we

[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2016-01-19 Thread Stefania (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15107722#comment-15107722 ] Stefania commented on CASSANDRA-9949: - Thanks, starting with the 3.0+ patch then. >

[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2016-01-18 Thread Marcus Eriksson (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15105471#comment-15105471 ] Marcus Eriksson commented on CASSANDRA-9949: The approach looks good to me, two comments;

[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2016-01-18 Thread Stefania (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15106398#comment-15106398 ] Stefania commented on CASSANDRA-9949: - Thank you for your review [~krummas], please see [this

[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2016-01-13 Thread Stefania (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15096157#comment-15096157 ] Stefania commented on CASSANDRA-9949: - The patch for 2.2 is ready, I would like a review before I

[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2016-01-13 Thread Stefania (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15096194#comment-15096194 ] Stefania commented on CASSANDRA-9949: - There is something really weird with the [first dtest

[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2016-01-13 Thread Philip Thompson (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15096294#comment-15096294 ] Philip Thompson commented on CASSANDRA-9949: It looks like the second run is working now.

[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2016-01-13 Thread Stefania (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15096299#comment-15096299 ] Stefania commented on CASSANDRA-9949: - Thanks. > maxPurgeableTimestamp needs to check memtables

[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2016-01-11 Thread Stefania (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15092139#comment-15092139 ] Stefania commented on CASSANDRA-9949: - Thanks for clarifying the fix version. Re. performance

[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2016-01-06 Thread Aleksey Yeschenko (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15086347#comment-15086347 ] Aleksey Yeschenko commented on CASSANDRA-9949: -- [~Stefania] I would stick to 2.2.x+ >

[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2016-01-06 Thread Aleksey Yeschenko (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15086350#comment-15086350 ] Aleksey Yeschenko commented on CASSANDRA-9949: -- I also wonder what the effect on

[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2016-01-04 Thread Stefania (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15081334#comment-15081334 ] Stefania commented on CASSANDRA-9949: - [~iamaleksey] : can you confirm this is required on all

[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2015-07-31 Thread Aleksey Yeschenko (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14649971#comment-14649971 ] Aleksey Yeschenko commented on CASSANDRA-9949: -- Very out of order, or

[jira] [Commented] (CASSANDRA-9949) maxPurgeableTimestamp needs to check memtables too

2015-07-31 Thread Jonathan Ellis (JIRA)
[ https://issues.apache.org/jira/browse/CASSANDRA-9949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14649931#comment-14649931 ] Jonathan Ellis commented on CASSANDRA-9949: --- Nit: should probably reverse