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

stack commented on HBASE-12133:
-------------------------------

updateMin/Max is fine where it is.

I'd just prefer that we not have multiple histogram implementations if possible 
or if we have two, that it is clear when one should use one over the other.  
You have code coming in that needs this implementation?

> Add FastLongHistogram for metric computation
> --------------------------------------------
>
>                 Key: HBASE-12133
>                 URL: https://issues.apache.org/jira/browse/HBASE-12133
>             Project: HBase
>          Issue Type: New Feature
>          Components: metrics
>    Affects Versions: 0.98.8
>            Reporter: Yi Deng
>            Assignee: Yi Deng
>            Priority: Minor
>              Labels: histogram, metrics
>             Fix For: 0.98.8
>
>         Attachments: 
> 0001-Add-FastLongHistogram-for-fast-histogram-estimation.patch, 
> 0001-Add-FastLongHistogram-for-fast-histogram-estimation.patch, 
> 0001-Add-FastLongHistogram-for-fast-histogram-estimation.patch
>
>
> FastLongHistogram is a thread-safe class that estimate distribution of data 
> and computes the quantiles. It's useful for computing aggregated metrics like 
> P99/P95.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to