rmdmattingly commented on code in PR #4937:
URL: https://github.com/apache/hbase/pull/4937#discussion_r1065148860
##########
hbase-client/src/main/java/org/apache/hadoop/hbase/shaded/protobuf/ProtobufUtil.java:
##########
@@ -1072,6 +1077,22 @@ public static ClientProtos.Scan toScan(final Scan scan)
throws IOException {
return scanBuilder.build();
}
+ public static List<Operation> toMulti(MultiRequest multiRequest) {
Review Comment:
> it's missing hasServiceCall() handling
We took this conversation offline and decided not to support service calls
here, primarily because the `RegionCoprocessServiceExec` class [is not meant to
be
instantiated](https://github.com/apache/hbase/blob/9573466112712ea5823ddc48ebb3a084ff3e60e4/hbase-client/src/main/java/org/apache/hadoop/hbase/client/RegionCoprocessorServiceExec.java#L32-L33)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
