You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@knox.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2023/08/30 20:38:00 UTC
[jira] [Work logged] (KNOX-2953) Uniform name of http method of client metric to lowerCase
[ https://issues.apache.org/jira/browse/KNOX-2953?focusedWorklogId=879208&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-879208 ]
ASF GitHub Bot logged work on KNOX-2953:
----------------------------------------
Author: ASF GitHub Bot
Created on: 30/Aug/23 20:37
Start Date: 30/Aug/23 20:37
Worklog Time Spent: 10m
Work Description: moresandeep commented on PR #789:
URL: https://github.com/apache/knox/pull/789#issuecomment-1699805105
Thank you for the patch @a516072575
Issue Time Tracking
-------------------
Worklog Id: (was: 879208)
Remaining Estimate: 0h
Time Spent: 10m
> Uniform name of http method of client metric to lowerCase
> ---------------------------------------------------------
>
> Key: KNOX-2953
> URL: https://issues.apache.org/jira/browse/KNOX-2953
> Project: Apache Knox
> Issue Type: Improvement
> Components: Metrics
> Reporter: liang.feng
> Priority: Minor
> Attachments: gateway-client-metric.png, gateway-metric-client-after-fix.png
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> The metric name starting with client should set httpMethod to lowercase, when build metric name, just like the service metric name at [#
> [https://github.com/apache/knox/blob/master/gateway-server/src/main/java/org/apache/knox/gateway/services/metrics/impl/instr/InstrHttpClientBuilderProvider.java#L68]]
>
> There are many similar metric names now,due to name of httpMethod is non-uniform.
> For example:
> When I query from HBase with curl command , The command curl -X Get 和 curl -X GET will generate two similar metric name
> !gateway-client-metric.png!
> We should uniformly change the method name to lowercase so that the metric names of service and client are unified.
>
> The effect after applying the [PR789|https://github.com/apache/knox/pull/789]
> !gateway-metric-client-after-fix.png!
--
This message was sent by Atlassian Jira
(v8.20.10#820010)