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

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

Firmly convinced that trying to load tmp is misguided.  At best it will be 
misleading to someone reading the code.  The odds of a failure such that 
loading the tmp will be useful are literally negligible.

Assuming we cut this back to loading the old file, does this do the Right Thing 
if sstables are mentioned in the manifest that no longer actually exist?
                
> 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
>
>
> 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