Repository: hbase Updated Branches: refs/heads/master ef7f8467a -> f34011860
HBASE-14685 Procedure Id is not set for MasterRpcServices#modifyTable Signed-off-by: Matteo Bertozzi <matteo.berto...@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/master Commit: f34011860e04fd1261b3b30cab0265016a22faa2 Parents: ef7f846 Author: Ashish Singhi <ashish.sin...@huawei.com> Authored: Fri Oct 23 23:11:35 2015 +0530 Committer: Matteo Bertozzi <matteo.berto...@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