You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by bi...@apache.org on 2020/04/07 02:38:34 UTC

[hbase] branch branch-2.2 updated: HBASE-24077 When encounter RowTooBigException, log the row info. (#1379)

This is an automated email from the ASF dual-hosted git repository.

binlijin pushed a commit to branch branch-2.2
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2.2 by this push:
     new 3d061a8  HBASE-24077 When encounter RowTooBigException, log the row info. (#1379)
3d061a8 is described below

commit 3d061a84805458af71b3a8a817e32019926d2a7f
Author: binlijin <bi...@gmail.com>
AuthorDate: Tue Apr 7 10:28:35 2020 +0800

    HBASE-24077 When encounter RowTooBigException, log the row info. (#1379)
    
    Signed-off-by: Viraj Jasani <vj...@apache.org>
    Signed-off-by: Pankaj <pa...@apache.org>
    Signed-off-by: Jan Hentschel <ja...@ultratendency.com>
---
 .../java/org/apache/hadoop/hbase/regionserver/StoreScanner.java  | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

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 725d8e6..654158e 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
@@ -635,8 +635,13 @@ public class StoreScanner extends NonReversedNonLazyKeyValueScanner
             scannerContext.incrementBatchProgress(1);
 
             if (matcher.isUserScan() && totalBytesRead > maxRowSize) {
-              throw new RowTooBigException(
-                  "Max row size allowed: " + maxRowSize + ", but the row is bigger than that.");
+              String message = "Max row size allowed: " + maxRowSize
+                + ", but the row is bigger than that, the row info: " + CellUtil
+                .toString(cell, false) + ", already have process row cells = " + outResult.size()
+                + ", it belong to region = " + store.getHRegion().getRegionInfo()
+                .getRegionNameAsString();
+              LOG.warn(message);
+              throw new RowTooBigException(message);
             }
           }