You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by mb...@apache.org on 2016/05/13 22:08:39 UTC
[5/5] hbase git commit: HBASE-13532 Make UnknownScannerException less
scary by giving more information in the exception string. (Apekshit)
HBASE-13532 Make UnknownScannerException less scary by giving more information in the exception string. (Apekshit)
Signed-off-by: Matteo Bertozzi <ma...@cloudera.com>
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/535e7672
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/535e7672
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/535e7672
Branch: refs/heads/branch-1.1
Commit: 535e767296685529cc04d7346690dcd8ac7b8604
Parents: 51baea6
Author: Apekshit <ap...@gmail.com>
Authored: Tue May 10 19:11:51 2016 -0700
Committer: Matteo Bertozzi <ma...@cloudera.com>
Committed: Fri May 13 15:07:15 2016 -0700
----------------------------------------------------------------------
.../org/apache/hadoop/hbase/regionserver/RSRpcServices.java | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/535e7672/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java
index 318005b..b4158d1 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java
@@ -2262,9 +2262,14 @@ public class RSRpcServices implements HBaseRPCErrorHandler,
if (request.hasScannerId()) {
rsh = scanners.get(scannerName);
if (rsh == null) {
- LOG.info("Client tried to access missing scanner " + scannerName);
+ LOG.warn("Client tried to access missing scanner " + scannerName);
throw new UnknownScannerException(
- "Name: " + scannerName + ", already closed?");
+ "Unknown scanner '" + scannerName + "'. This can happen due to any of the following "
+ + "reasons: a) Scanner id given is wrong, b) Scanner lease expired because of "
+ + "long wait between consecutive client checkins, c) Server may be closing down, "
+ + "d) RegionServer restart during upgrade.\nIf the issue is due to reason (b), a "
+ + "possible fix would be increasing the value of"
+ + "'hbase.client.scanner.timeout.period' configuration.");
}
scanner = rsh.s;
HRegionInfo hri = scanner.getRegionInfo();