You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ya...@apache.org on 2017/07/25 07:16:53 UTC

hbase git commit: HBASE-15968 (addendum) revert unrelated PE changing

Repository: hbase
Updated Branches:
  refs/heads/master 1ac4152b1 -> e1cd59bbc


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/master
Commit: e1cd59bbc4c370375ad71aabfcc02b16c46e5ba2
Parents: 1ac4152
Author: Phil Yang <ya...@apache.org>
Authored: Tue Jul 25 15:16:02 2017 +0800
Committer: Phil Yang <ya...@apache.org>
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);
     }