cassandra-pr mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From eperott <>
Subject [GitHub] cassandra pull request #:
Date Thu, 06 Sep 2018 14:19:04 GMT
Github user eperott commented on the pull request:
    In src/java/org/apache/cassandra/metrics/
    In src/java/org/apache/cassandra/metrics/ on line
    If I remember, reason for simply returning size of bucket array was that most (if not
all) implementations in the metrics library was actually returning a value which represented
the size of the container holding samples rather than the number of recorded values. So was
just trying to stay consistent with that. A better option would perhaps have been to return
the number of samples being considered in the snapshot, but that would have been non-trivial
(I think) when taking the forward decay into account. Changing behavior here would possibly
have side effects within the metrics library, but I will have to check a little more to tell.
    I'm not sure I understand you comment wrt count? Are you saying all snapshots get a count()
of zero?


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message