[ https://issues.apache.org/jira/browse/HBASE-9612?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13782205#comment-13782205 ]
Hudson commented on HBASE-9612: ------------------------------- FAILURE: Integrated in hbase-0.96-hadoop2 #65 (See [https://builds.apache.org/job/hbase-0.96-hadoop2/65/]) HBASE-9612 Ability to batch edits destined to different regions (stack: rev 1527687) * /hbase/branches/0.96/hbase-client/src/main/java/org/apache/hadoop/hbase/client/ClientSmallScanner.java * /hbase/branches/0.96/hbase-client/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java * /hbase/branches/0.96/hbase-client/src/main/java/org/apache/hadoop/hbase/client/HTable.java * /hbase/branches/0.96/hbase-client/src/main/java/org/apache/hadoop/hbase/client/MultiServerCallable.java * /hbase/branches/0.96/hbase-client/src/main/java/org/apache/hadoop/hbase/client/ScannerCallable.java * /hbase/branches/0.96/hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/PayloadCarryingRpcController.java * /hbase/branches/0.96/hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/RpcClient.java * /hbase/branches/0.96/hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ProtobufUtil.java * /hbase/branches/0.96/hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/RequestConverter.java * /hbase/branches/0.96/hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ResponseConverter.java * /hbase/branches/0.96/hbase-protocol/README.txt * /hbase/branches/0.96/hbase-protocol/src/main/java/org/apache/hadoop/hbase/protobuf/generated/ClientProtos.java * /hbase/branches/0.96/hbase-protocol/src/main/java/org/apache/hadoop/hbase/protobuf/generated/RPCProtos.java * /hbase/branches/0.96/hbase-protocol/src/main/protobuf/Client.proto * /hbase/branches/0.96/hbase-protocol/src/main/protobuf/RPC.proto * /hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java * /hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java * /hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/QosFunction.java * /hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALEditsReplaySink.java * /hbase/branches/0.96/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestQosFunction.java > Ability to batch edits destined to different regions > ---------------------------------------------------- > > Key: HBASE-9612 > URL: https://issues.apache.org/jira/browse/HBASE-9612 > Project: HBase > Issue Type: Bug > Affects Versions: 0.95.0, 0.95.1, 0.95.2, 0.96.0 > Reporter: Benoit Sigoure > Assignee: stack > Priority: Critical > Fix For: 0.98.0, 0.96.0 > > Attachments: 9612.096.v5.txt, 9612v2.txt, 9612v3.txt, 9612v4.txt, > 9612v5.txt, 9612.wip.txt > > > The old (pre-PB) "multi" and "multiPut" RPCs allowed one to batch edits > destined to different regions. Seems like we've lost this ability after the > switch to protobufs. > The {{MultiRequest}} only contains one {{RegionSpecifier}}, and a list of > {{MultiAction}}. The {{MultiAction}} message is contains either a single > {{MutationProto}} or a {{Get}} (but not both – so its name is misleading as > there is nothing "multi" about it). Also it seems redundant with > {{MultiGetRequest}}, I'm not sure what's the point of supporting {{Get}} in > {{MultiAction}}. > I propose that we change {{MultiRequest}} to be a just a list of > {{MultiAction}}, and {{MultiAction}} will contain the {{RegionSpecifier}}, > the {{bool atomic}} and a list of {{MutationProto}}. This would be a > non-backward compatible protobuf change. > If we want we can support mixing edits and reads, in which case we'd also add > a list of {{Get}} in {{MultiAction}}, and we'd have support having both that > list and the list of {{MutationProto}} set at the same time. But this is a > bonus and can be done later (in a backward compatible manner, hence no need > to rush on this one). -- This message was sent by Atlassian JIRA (v6.1#6144)