Github user aweisberg commented on the pull request:

    
https://github.com/apache/cassandra/commit/2e90259669bddf04b6c1dba38b604aa6a33dcd47#commitcomment-30407175
  
    In 
src/java/org/apache/cassandra/metrics/DecayingEstimatedHistogramReservoir.java:
    In 
src/java/org/apache/cassandra/metrics/DecayingEstimatedHistogramReservoir.java 
on line 419:
    @eperott Hey wondering why this is done this way? It seems like the correct 
value is pre-calculated? I checked and didn't see any usages of this size 
method anywhere in C*.
    
    I want to change it to return count so I can check for empty histograms 
that return bogus values for P99 because they are empty.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to