hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From apurt...@apache.org
Subject [2/2] hbase git commit: Amend HBASE-13303 Fix size calculation of results on the region server
Date Sat, 28 Mar 2015 05:26:38 GMT
Amend HBASE-13303 Fix size calculation of results on the region server

Signed-off-by: Andrew Purtell <apurtell@apache.org>


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

Branch: refs/heads/branch-1
Commit: 7da7a77ff970b23c52106ef95520543dccf9d0c5
Parents: 4bda365
Author: Jonathan Lawlor <jonathan.lawlor@cloudera.com>
Authored: Thu Mar 26 15:54:38 2015 -0700
Committer: Andrew Purtell <apurtell@apache.org>
Committed: Fri Mar 27 22:12:07 2015 -0700

----------------------------------------------------------------------
 .../main/java/org/apache/hadoop/hbase/regionserver/HRegion.java    | 2 +-
 .../java/org/apache/hadoop/hbase/regionserver/StoreScanner.java    | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/7da7a77f/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
index 9233275..1943c3a 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
@@ -5664,7 +5664,7 @@ public class HRegion implements HeapSize, PropagatingConfigurationObserver
{ //
 
       long size = 0;
       for (Cell c : results) {
-        size += CellUtil.estimatedHeapSizeOf(c);
+        size += CellUtil.estimatedHeapSizeOfWithoutTags(c);
       }
 
       return size;

http://git-wip-us.apache.org/repos/asf/hbase/blob/7da7a77f/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
index 7ce4e0b..298d5bc 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
@@ -544,7 +544,7 @@ public class StoreScanner extends NonReversedNonLazyKeyValueScanner
             outResult.add(cell);
             count++;
             totalBytesRead += CellUtil.estimatedSerializedSizeOf(cell);
-            totalHeapSize += CellUtil.estimatedHeapSizeOf(cell);
+            totalHeapSize += CellUtil.estimatedHeapSizeOfWithoutTags(cell);
             if (totalBytesRead > maxRowSize) {
               throw new RowTooBigException("Max row size allowed: " + maxRowSize
               + ", but the row is bigger than that.");


Mime
View raw message