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

Branimir Lambov commented on CASSANDRA-12802:
---------------------------------------------

Code LGTM

[~zznate]: I would say that this patch removes risk as it replaces racy logic 
in {{recover}} with a very simple filter based on a value that:
- was there before the patch;
- could cause serious trouble if it was wrong regardless of the patch.

> RecoveryManagerTruncateTest fails in 2.1
> ----------------------------------------
>
>                 Key: CASSANDRA-12802
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-12802
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Stefania
>            Assignee: Stefania
>             Fix For: 2.1.x
>
>
> Example failure:
> http://cassci.datastax.com/view/cassandra-2.1/job/cassandra-2.1_testall/388/testReport/junit/org.apache.cassandra.db/RecoveryManagerTruncateTest/testTruncatePointInTime/



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to