HBASE-15968 (addendum) revert unrelated PE changing

Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/e1cd59bb
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/e1cd59bb
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/e1cd59bb

Branch: refs/heads/HBASE-18426
Commit: e1cd59bbc4c370375ad71aabfcc02b16c46e5ba2
Parents: 1ac4152
Author: Phil Yang <[email protected]>
Authored: Tue Jul 25 15:16:02 2017 +0800
Committer: Phil Yang <[email protected]>
Committed: Tue Jul 25 15:16:02 2017 +0800

----------------------------------------------------------------------
 .../test/java/org/apache/hadoop/hbase/PerformanceEvaluation.java  | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/e1cd59bb/hbase-server/src/test/java/org/apache/hadoop/hbase/PerformanceEvaluation.java
----------------------------------------------------------------------
diff --git 
a/hbase-server/src/test/java/org/apache/hadoop/hbase/PerformanceEvaluation.java 
b/hbase-server/src/test/java/org/apache/hadoop/hbase/PerformanceEvaluation.java
index af0956a..cabbdb4 100644
--- 
a/hbase-server/src/test/java/org/apache/hadoop/hbase/PerformanceEvaluation.java
+++ 
b/hbase-server/src/test/java/org/apache/hadoop/hbase/PerformanceEvaluation.java
@@ -1064,7 +1064,7 @@ public class PerformanceEvaluation extends Configured 
implements Tool {
     }
 
     void updateValueSize(final Result r) throws IOException {
-      if (r == null ) return;
+      if (r == null || !isRandomValueSize()) return;
       int size = 0;
       for (CellScanner scanner = r.cellScanner(); scanner.advance();) {
         size += scanner.current().getValueLength();
@@ -1073,6 +1073,7 @@ public class PerformanceEvaluation extends Configured 
implements Tool {
     }
 
     void updateValueSize(final int valueSize) {
+      if (!isRandomValueSize()) return;
       this.valueSizeHistogram.update(valueSize);
     }
 

Reply via email to