[jira] [Commented] (ZOOKEEPER-2979) Use dropwizard library histogram for proposal-related metrics

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ZOOKEEPER-2979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16449004#comment-16449004 ] 

ASF GitHub Bot commented on ZOOKEEPER-2979:
-------------------------------------------

Github user phunt commented on the issue:

    https://github.com/apache/zookeeper/pull/440
  
    @anmolnar I believe you misunderstood my comment above:
    
    > likely this change will go into master (3.6.0+) - impact on backports. I'm assuming we would want to take advantage of this library across the board. (can we do that? I assume yes?)
    
    > Sure, I'll have to backport it to branch-3.4, it will need some manual work for sure, but I don't expect it to be the end of the world.
    
    I meant that I was planning to only commit this change for 3.6.0+ - I am generally reticent to add a new dependency so late in the 3.5 train. Do you feel strongly otw?


> Use dropwizard library histogram for proposal-related metrics
> -------------------------------------------------------------
>
>                 Key: ZOOKEEPER-2979
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2979
>             Project: ZooKeeper
>          Issue Type: Sub-task
>          Components: server
>            Reporter: Andor Molnar
>            Assignee: Andor Molnar
>            Priority: Major
>
> This Jira in intended to be the successor of ZOOKEEPER-2939.
> By using dropwizard library's Histogram component we'll be able to provide more sophisticated statistics on Proposal sizes.
> From the docs:
> "A histogram measures the statistical distribution of values in a stream of data. In addition to minimum, maximum, mean, etc., it also measures median, 75th, 90th, 95th, 98th, 99th, and 99.9th percentiles."
> [http://metrics.dropwizard.io/3.1.0/manual/core/#histograms]



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)