[
https://issues.apache.org/jira/browse/HBASE-27553?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17746788#comment-17746788
]
Hudson commented on HBASE-27553:
--------------------------------
Results for branch branch-2
[build #851 on
builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/851/]:
(x) *{color:red}-1 overall{color}*
----
details (if available):
(/) {color:green}+1 general checks{color}
-- For more information [see general
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/851/General_20Nightly_20Build_20Report/]
(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2)
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/851/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/]
(x) {color:red}-1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3)
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/851/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]
(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11
report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2/851/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/]
(/) {color:green}+1 source release artifact{color}
-- See build output for details.
(/) {color:green}+1 client integration test{color}
> 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
> Fix For: 2.6.0, 2.4.18, 2.5.6, 3.0.0-beta-1
>
>
> 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)
