metrics: EstimatedHistogram::getValues() returns bucketOffsets

scylladb / scylla-jmx

Scylla JMX proxy

GNU Affero General Public License v3.0

28 stars 51 forks source link

Closed Deexie closed 1 year ago

Deexie commented 1 year ago

EstimatedHistogram is expected to return number of sstables as its value, but it returns number of sstables reads.

EstimatedHistogram::getValues() returns bucketOffsets which contains the number of sstables.

Fixes: #199.

bhalevy commented 1 year ago

LGTM.

Is there a test to verify the fix?

bhalevy commented 1 year ago