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

Enis Soztutar commented on HBASE-11815:
---------------------------------------

+1 from me as well. 

> Flush and compaction could just close the tmp writer if there is an exception
> -----------------------------------------------------------------------------
>
>                 Key: HBASE-11815
>                 URL: https://issues.apache.org/jira/browse/HBASE-11815
>             Project: HBase
>          Issue Type: Bug
>            Reporter: ramkrishna.s.vasudevan
>            Assignee: ramkrishna.s.vasudevan
>            Priority: Trivial
>             Fix For: 2.0.0, 0.98.7, 0.99.1
>
>         Attachments: HBASE-11815.patch, HBASE-11815_1.patch
>
>
> A minor change.
> {code}
>         try {
>           flushed = performFlush(scanner, writer, smallestReadPoint);
>         } finally {
>           finalizeWriter(writer, cacheFlushId, status);
>         }
> {code}
> Whenever there is a failure during flush we should close the writer but 
> adding the meta data and setting the status would not be needed.
> {code}
>     status.setStatus("Flushing " + store + ": appending metadata");
>     writer.appendMetadata(cacheFlushSeqNum, false);
>     status.setStatus("Flushing " + store + ": closing flushed file");
>     writer.close();
> {code}



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

Reply via email to