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)