Bryan Beaudreault created HBASE-27553: -----------------------------------------
Summary: SlowLog does not include params for Mutations Key: HBASE-27553 URL: https://issues.apache.org/jira/browse/HBASE-27553 Project: HBase Issue Type: Bug Reporter: Bryan Beaudreault Assignee: Ray Mattingly SlowLog params are extracted via [ProtobufUtil.getSlowLogParams|https://github.com/apache/hbase/blob/master/hbase-client/src/main/java/org/apache/hadoop/hbase/shaded/protobuf/ProtobufUtil.java#L2154]. This method has various if/else branches for each request type, but mutation (the line linked above) is incorrect. Currently it handles MutationProto, but it should be MutateRequest. A MutationProto is never passed into this method, only MutateRequests so any MutateRequests being passed in now will fall through to the default case which contains nothing useful about the request. As part of fixing this, we should also ensure that we extract the region name from the MutateRequest to add into the SlowLogParams object like all the other requests. While we are here, the CoprocessorServiceRequest (handled further down) has a getRegion() method, but that is not passed into the SlowLogParams either. We should add that too. -- This message was sent by Atlassian Jira (v8.20.10#820010)