You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hbase.apache.org by jm...@apache.org on 2013/12/09 22:15:59 UTC

svn commit: r1549682 - /hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java

Author: jmhsieh
Date: Mon Dec  9 21:15:59 2013
New Revision: 1549682

URL: http://svn.apache.org/r1549682
Log:
HBASE-10108 NullPointerException thrown while using Canary with '-regionserver' option (Takeshi Miao)

Modified:
    hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java

Modified: hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java?rev=1549682&r1=1549681&r2=1549682&view=diff
==============================================================================
--- hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java (original)
+++ hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/tool/Canary.java Mon Dec  9 21:15:59 2013
@@ -568,6 +568,8 @@ public final class Canary implements Too
         return false;
       }
 
+      if (this.targets == null || this.targets.length == 0) return true;
+
       for (String target : this.targets) {
         for (TableName tableName : tableNames) {
           if (target.equals(tableName.getNameAsString())) {