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

Stu Hood commented on CASSANDRA-2468:
-------------------------------------

* SSTable.delete will throw an IOError on IOException, which might kill 
cleanupIfNecessary... should we consider having delete throw IOException? I'd 
prefer not to catch IOError.
* {{SSTable.tempComponentsFor}} could probably be merged with componentsFor and 
an enum {{LIVE}}, {{TEMP}} or {{LIVE | TEMP}}? Not a blocker, just sugar.

After the IOError is fixed, +1 from me. Thanks Aaron!

> Clean up after failed compaction
> --------------------------------
>
>                 Key: CASSANDRA-2468
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2468
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Core
>            Reporter: Jonathan Ellis
>            Assignee: Aaron Morton
>            Priority: Minor
>             Fix For: 0.7.7
>
>         Attachments: 
> 0001-clean-up-temp-files-after-failed-compaction-v08-2.patch, 
> 0001-clean-up-temp-files-after-failed-compaction-v08.patch, 
> 0001-cleanup-temp-files-after-failed-compaction-v07.patch
>
>
> (Started in CASSANDRA-2088.)

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to