Repository: hbase Updated Branches: refs/heads/branch-1 278c17d1e -> 92f415976 refs/heads/branch-1.1 51baea64d -> 535e76729 refs/heads/branch-1.2 6e3770b71 -> aaa436314 refs/heads/branch-1.3 0b59341d2 -> 71d453938 refs/heads/master 0e281e192 -> 60e19f60a
HBASE-13532 Make UnknownScannerException less scary by giving more information in the exception string. (Apekshit) Signed-off-by: Matteo Bertozzi <matteo.berto...@cloudera.com> Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/60e19f60 Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/60e19f60 Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/60e19f60 Branch: refs/heads/master Commit: 60e19f60a9767db6b145cf9884d2295412baffe0 Parents: 0e281e1 Author: Apekshit <apeksha...@gmail.com> Authored: Tue May 10 19:11:51 2016 -0700 Committer: Matteo Bertozzi <matteo.berto...@cloudera.com> Committed: Fri May 13 15:00:23 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/60e19f60/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 589c4ed..95ab36d 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 @@ -2542,9 +2542,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();
