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

Posted to commits@hbase.apache.org by jm...@apache.org on 2012/09/10 05:05:05 UTC

svn commit: r1382627 - /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestHBaseFsck.java

Author: jmhsieh
Date: Mon Sep 10 03:05:05 2012
New Revision: 1382627

URL: http://svn.apache.org/viewvc?rev=1382627&view=rev
Log:
HBASE-5631 ADDENDUM (extra comments)

Modified:
    hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestHBaseFsck.java

Modified: hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestHBaseFsck.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestHBaseFsck.java?rev=1382627&r1=1382626&r2=1382627&view=diff
==============================================================================
--- hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestHBaseFsck.java (original)
+++ hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestHBaseFsck.java Mon Sep 10 03:05:05 2012
@@ -432,7 +432,7 @@ public class TestHBaseFsck {
       HBaseFsck hbck = doFsck(conf, false); 
       assertErrors(hbck, new ERROR_CODE[] { ERROR_CODE.NO_TABLEINFO_FILE });
       
-      // fix OrphanTable with default .tableinfo
+      // fix OrphanTable with default .tableinfo (htd not yet cached on master)
       hbck = doFsck(conf, true);
       assertNoErrors(hbck);
       status = null;
@@ -447,7 +447,7 @@ public class TestHBaseFsck {
       fs.delete(status.getPath(), true);
       
       // fix OrphanTable with cache
-      htd = admin.getTableDescriptor(table.getBytes());
+      htd = admin.getTableDescriptor(table.getBytes()); // warms up cached htd on master
       hbck = doFsck(conf, true);
       assertNoErrors(hbck);
       status = null;