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

Jonathan Ellis commented on CASSANDRA-5041:
-------------------------------------------

Hmm, not sure what happened with v2, I'm getting

{noformat}
$ patch -p1 < ~/.JIRAClient/download/5041-v2.patch 
can't find file to patch at input line 10
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|Index: src/java/org/apache/cassandra/db/compaction/LeveledManifest.java
|IDEA additional info:
|Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
|<+>UTF-8
|Subsystem: com.intellij.openapi.diff.impl.patch.BaseRevisionTextPatchEP
|<+>/*\n * Licensed to the Apache Software Foundation (ASF) under one\n * or 
more contributor license a
{noformat}

Anecdotally, {{git diff}} tends to be more reliable than IDE diffing.
                
> Change LeveledManifest to try to load old and tmp json files
> ------------------------------------------------------------
>
>                 Key: CASSANDRA-5041
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-5041
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Carl Yeksigian
>              Labels: leveled
>             Fix For: 1.2.1
>
>         Attachments: 5041.txt, 5041-v2.patch
>
>
> Even though leveled compaction creates an old and a tmp file while writing 
> out a new file, the leveled manifest does not attempt to load those files.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to