You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@hbase.apache.org by "Ted Yu (JIRA)" <ji...@apache.org> on 2013/02/20 20:07:12 UTC

[jira] [Updated] (HBASE-7888) Simplify logic of SnapshotManager#getCompletedSnapshots()

     [ https://issues.apache.org/jira/browse/HBASE-7888?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ted Yu updated HBASE-7888:
--------------------------

    Summary: Simplify logic of SnapshotManager#getCompletedSnapshots()  (was: Simplify logic of getCompletedSnapshots())
    
> Simplify logic of SnapshotManager#getCompletedSnapshots()
> ---------------------------------------------------------
>
>                 Key: HBASE-7888
>                 URL: https://issues.apache.org/jira/browse/HBASE-7888
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Ted Yu
>
> This came up during code review of mega patch.
> isSnapshotCompleted() checks the existence of snapshotDir to see if the snapshot already exists.
> getCompletedSnapshots(), on the other hand, reads SnapshotDescriptionUtils.SNAPSHOTINFO_FILE for the same purpose.
> Since all the snapshot files are first created in a tmp dir and a rename is done to move them into the .snapshots dir upon completion, logic in getCompletedSnapshots() can be simplified.

--
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