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:49:00 UTC

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

     [ https://issues.apache.org/jira/browse/HBASE-18225?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Josh Elser updated HBASE-18225:
-------------------------------
    Attachment: HBASE-18225.001.patch

.001 trivial fix.

> 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
>
>         Attachments: HBASE-18225.001.patch
>
>
> 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)