[
https://issues.apache.org/jira/browse/HBASE-27553?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17720003#comment-17720003
]
Bryan Beaudreault commented on HBASE-27553:
-------------------------------------------
Yea I missed MutateRequest further down. But also, looks like both
MutateRequest and MutationProto aren't including row info (where GetRequest
does). We should take a look at each of these if branches to see there's
anything critical or useful to add.
> 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
> Priority: Minor
>
> 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)
