hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From apurt...@apache.org
Subject [1/4] hbase git commit: HBASE-13388 Handling NullPointer in ZKProcedureMemberRpcs while getting ZNode data (Vikas Vishwakarma)
Date Thu, 02 Apr 2015 21:24:47 GMT
Repository: hbase
Updated Branches:
  refs/heads/0.98 27ec6bc88 -> 2ec3443f0
  refs/heads/branch-1 3c0e3981a -> 6605dda7b
  refs/heads/branch-1.0 86cbd1b5d -> 374a29405
  refs/heads/master 578df6dfc -> 2365932d7


HBASE-13388 Handling NullPointer in ZKProcedureMemberRpcs while getting ZNode data (Vikas
Vishwakarma)


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/2365932d
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/2365932d
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/2365932d

Branch: refs/heads/master
Commit: 2365932d78180255be7145d8d247ee1f801aed1d
Parents: 578df6d
Author: Andrew Purtell <apurtell@apache.org>
Authored: Thu Apr 2 14:17:26 2015 -0700
Committer: Andrew Purtell <apurtell@apache.org>
Committed: Thu Apr 2 14:17:26 2015 -0700

----------------------------------------------------------------------
 .../org/apache/hadoop/hbase/procedure/ZKProcedureMemberRpcs.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/2365932d/hbase-server/src/main/java/org/apache/hadoop/hbase/procedure/ZKProcedureMemberRpcs.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/procedure/ZKProcedureMemberRpcs.java
b/hbase-server/src/main/java/org/apache/hadoop/hbase/procedure/ZKProcedureMemberRpcs.java
index a13292e..387c2e4 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/procedure/ZKProcedureMemberRpcs.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/procedure/ZKProcedureMemberRpcs.java
@@ -200,7 +200,6 @@ public class ZKProcedureMemberRpcs implements ProcedureMemberRpcs {
     Subprocedure subproc = null;
     try {
       byte[] data = ZKUtil.getData(zkController.getWatcher(), path);
-      LOG.debug("start proc data length is " + data.length);
       if (!ProtobufUtil.isPBMagicPrefix(data)) {
         String msg = "Data in for starting procuedure " + opName +
           " is illegally formatted (no pb magic). " +
@@ -208,6 +207,7 @@ public class ZKProcedureMemberRpcs implements ProcedureMemberRpcs {
         LOG.error(msg);
         throw new IllegalArgumentException(msg);
       }
+      LOG.debug("start proc data length is " + data.length);
       data = Arrays.copyOfRange(data, ProtobufUtil.lengthOfPBMagic(), data.length);
       LOG.debug("Found data for znode:" + path);
       subproc = member.createSubprocedure(opName, data);


Mime
View raw message