You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2013/09/13 23:34:26 UTC

svn commit: r1523113 - /hbase/branches/0.96/hbase-server/src/test/java/org/apache/hadoop/hbase/HFilePerformanceEvaluation.java

Author: stack
Date: Fri Sep 13 21:34:25 2013
New Revision: 1523113

URL: http://svn.apache.org/r1523113
Log:
HBASE-7953 Some HFilePerformanceEvaluation tests might fail because of scanner.getValue when there is no more row

Modified:
    hbase/branches/0.96/hbase-server/src/test/java/org/apache/hadoop/hbase/HFilePerformanceEvaluation.java

Modified: hbase/branches/0.96/hbase-server/src/test/java/org/apache/hadoop/hbase/HFilePerformanceEvaluation.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.96/hbase-server/src/test/java/org/apache/hadoop/hbase/HFilePerformanceEvaluation.java?rev=1523113&r1=1523112&r2=1523113&view=diff
==============================================================================
--- hbase/branches/0.96/hbase-server/src/test/java/org/apache/hadoop/hbase/HFilePerformanceEvaluation.java (original)
+++ hbase/branches/0.96/hbase-server/src/test/java/org/apache/hadoop/hbase/HFilePerformanceEvaluation.java Fri Sep 13 21:34:25 2013
@@ -284,7 +284,10 @@ public class HFilePerformanceEvaluation 
     void doRow(int i) throws Exception {
       HFileScanner scanner = this.reader.getScanner(false, true);
       byte [] b = getRandomRow();
-      scanner.seekTo(b);
+      if (scanner.seekTo(b) < 0) {
+        LOG.info("Not able to seekTo " + new String(b));
+        return;
+      }
       ByteBuffer k = scanner.getKey();
       PerformanceEvaluationCommons.assertKey(b, k);
       ByteBuffer v = scanner.getValue();
@@ -309,7 +312,7 @@ public class HFilePerformanceEvaluation 
       HFileScanner scanner = this.reader.getScanner(false, false);
       byte [] b = getRandomRow();
       if (scanner.seekTo(b) != 0) {
-        System.out.println("Nonexistent row: " + new String(b));
+        LOG.info("Nonexistent row: " + new String(b));
         return;
       }
       ByteBuffer k = scanner.getKey();
@@ -317,7 +320,8 @@ public class HFilePerformanceEvaluation 
       // System.out.println("Found row: " + new String(b));
       for (int ii = 0; ii < 30; ii++) {
         if (!scanner.next()) {
-          System.out.println("NOTHING FOLLOWS");
+          LOG.info("NOTHING FOLLOWS");
+          return;
         }
         ByteBuffer v = scanner.getValue();
         PerformanceEvaluationCommons.assertValueSize(v.limit(), ROW_LENGTH);
@@ -341,10 +345,15 @@ public class HFilePerformanceEvaluation 
     @Override
     void doRow(int i) throws Exception {
       HFileScanner scanner = this.reader.getScanner(false, true);
-      scanner.seekTo(getGaussianRandomRowBytes());
+      byte[] gaussianRandomRowBytes = getGaussianRandomRowBytes();
+      if (scanner.seekTo(gaussianRandomRowBytes < 0) {
+        LOG.info("Not able to seekTo " + new String(gaussianRandomRowBytes));
+        return;
+      }
       for (int ii = 0; ii < 30; ii++) {
         if (!scanner.next()) {
-          System.out.println("NOTHING FOLLOWS");
+          LOG.info("NOTHING FOLLOWS");
+          return;
         }
         scanner.getKey();
         scanner.getValue();