You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ns...@apache.org on 2011/09/27 03:07:32 UTC

svn commit: r1176152 - /hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestLogRolling.java

Author: nspiegelberg
Date: Tue Sep 27 01:07:32 2011
New Revision: 1176152

URL: http://svn.apache.org/viewvc?rev=1176152&view=rev
Log:
Merged unit test changes from 0.20 to trunk

Modified:
    hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestLogRolling.java

Modified: hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestLogRolling.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestLogRolling.java?rev=1176152&r1=1176151&r2=1176152&view=diff
==============================================================================
--- hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestLogRolling.java (original)
+++ hbase/branches/0.89/src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestLogRolling.java Tue Sep 27 01:07:32 2011
@@ -261,7 +261,7 @@ public class TestLogRolling extends HBas
                 getPipeline != null);
     Object repl = getPipeline.invoke(stm, new Object []{} /*NO_ARGS*/);
     DatanodeInfo[] pipeline = (DatanodeInfo[]) repl;
-    assertTrue(pipeline.length == fs.getDefaultReplication());
+    assertEquals(fs.getDefaultReplication(), pipeline.length);
     DataNodeProperties dnprop = dfsCluster.stopDataNode(pipeline[0].getName());
     assertTrue(dnprop != null);
 
@@ -275,7 +275,6 @@ public class TestLogRolling extends HBas
     writeData(table, 3);
     assertTrue("The log should not roll again.", 
               log.getFilenum() == newFilenum);
-    assertTrue("New log file should have the default replication",
-              log.getLogReplication() == fs.getDefaultReplication());
+    assertEquals(fs.getDefaultReplication(), log.getLogReplication());
   }
 }