You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by sy...@apache.org on 2015/10/24 00:50:09 UTC

[49/50] [abbrv] hbase git commit: HBASE-14685 Procedure Id is not set for MasterRpcServices#modifyTable

HBASE-14685 Procedure Id is not set for MasterRpcServices#modifyTable

Signed-off-by: Matteo Bertozzi <ma...@cloudera.com>


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

Branch: refs/heads/hbase-12439
Commit: f34011860e04fd1261b3b30cab0265016a22faa2
Parents: ef7f846
Author: Ashish Singhi <as...@huawei.com>
Authored: Fri Oct 23 23:11:35 2015 +0530
Committer: Matteo Bertozzi <ma...@cloudera.com>
Committed: Fri Oct 23 11:37:10 2015 -0700

----------------------------------------------------------------------
 .../java/org/apache/hadoop/hbase/master/MasterRpcServices.java   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/f3401186/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterRpcServices.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterRpcServices.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterRpcServices.java
index f5dcd0e..b269c3d 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterRpcServices.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterRpcServices.java
@@ -1214,15 +1214,15 @@ public class MasterRpcServices extends RSRpcServices
   public ModifyTableResponse modifyTable(RpcController controller,
       ModifyTableRequest req) throws ServiceException {
     try {
-      master.modifyTable(
+      long procId = master.modifyTable(
         ProtobufUtil.toTableName(req.getTableName()),
         HTableDescriptor.convert(req.getTableSchema()),
         req.getNonceGroup(),
         req.getNonce());
+      return ModifyTableResponse.newBuilder().setProcId(procId).build();
     } catch (IOException ioe) {
       throw new ServiceException(ioe);
     }
-    return ModifyTableResponse.newBuilder().build();
   }
 
   @Override