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

Hudson commented on HBASE-8413:
-------------------------------

Integrated in HBase-0.94 #965 (See 
[https://builds.apache.org/job/HBase-0.94/965/])
    HBASE-8413 Snapshot verify region will always fail if the HFile has been 
archived (Jerry He) (Revision 1471323)

     Result = FAILURE
mbertozzi : 
Files : 
* 
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/master/snapshot/MasterSnapshotVerifier.java

                
> Snapshot verify region will always fail if the HFile has been archived
> ----------------------------------------------------------------------
>
>                 Key: HBASE-8413
>                 URL: https://issues.apache.org/jira/browse/HBASE-8413
>             Project: HBase
>          Issue Type: Bug
>          Components: snapshots
>    Affects Versions: 0.94.6, 0.95.0
>            Reporter: Jerry He
>            Assignee: Jerry He
>             Fix For: 0.98.0, 0.94.8, 0.95.2
>
>         Attachments: HBASE-8413-trunk.patch
>
>
> MasterSnapshotVerifier::verifyRegion():
> ...
> // check to see if hfile is present in the real table
>         String fileName = hfile.getPath().getName();
>         Path file = new Path(realCfDir, fileName);
>         Path archived = new Path(archivedCfDir, fileName);
>         if (!fs.exists(file) && !file.equals(archived)) {
>           throw new CorruptedSnapshotException("Can't find hfile: " + 
> hfile.getPath()
>               + " in the real (" + realCfDir + ") or archive (" + 
> archivedCfDir
>               + ") directory for the primary table.", snapshot);
>         }
> ....
> The second part of the if condition will always be true:
> !file.equals(archived)
> Corrected:
> if (!fs.exists(file) && !fs.exists(archived))

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