You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Josh Elser (JIRA)" <ji...@apache.org> on 2017/06/16 00:42:00 UTC

[jira] [Created] (HBASE-18225) Fix findbugs regression calling toString() on an array

Josh Elser created HBASE-18225:
----------------------------------

             Summary: Fix findbugs regression calling toString() on an array
                 Key: HBASE-18225
                 URL: https://issues.apache.org/jira/browse/HBASE-18225
             Project: HBase
          Issue Type: Bug
            Reporter: Josh Elser
            Assignee: Josh Elser
            Priority: Trivial
             Fix For: 2.0.0, 3.0.0


Looks like we got a findbugs warning as a result of HBASE-18166

{code}
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 1d04944250..b7e0244aa2 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
@@ -2807,8 +2807,8 @@ public class RSRpcServices implements HBaseRPCErrorHandler,
     HRegionInfo hri = rsh.s.getRegionInfo();
     // Yes, should be the same instance
     if (regionServer.getOnlineRegion(hri.getRegionName()) != rsh.r) {
-      String msg = "Region was re-opened after the scanner" + scannerName + " was created: "
-          + hri.getRegionNameAsString();
+      String msg = "Region has changed on the scanner " + scannerName + ": regionName="
+          + hri.getRegionName() + ", scannerRegionName=" + rsh.r;
{code}

Looks like {{hri.getRegionNameAsString()}} was unintentionally changed to {{hri.getRegionName()}}, [~syuanjiang]/[~stack]?



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)