1247260 - Histogram bucket limit should allow for n_values 100 for enumerated histograms

Status: RESOLVED FIXED

(Toolkit :: Telemetry, defect, P5)

Description

[Georg Fritzsche [:gfritzsche]]

We add +1 to the bucket count for enumerated histograms here:
https://dxr.mozilla.org/mozilla-central/rev/2dfb45d74f42d2a0010696f5fd47c7a7da94cedb/toolkit/components/telemetry/histogram_tools.py#287

We check for the bucket count not exceeding 100 here:
https://dxr.mozilla.org/mozilla-central/rev/2dfb45d74f42d2a0010696f5fd47c7a7da94cedb/toolkit/components/telemetry/histogram_tools.py#270

For the other histogram the limit works as expected, but with enumerated histograms the limit triggers with n_values:100 instead of from 101 on.

Comment 1

[Beatriz Rizental [:brizental]]

Attachment: Bug 1247260 - Let enumerated and categorical histograms have 100 buckets. r=chutten,janerik

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3eb9521efab1
Let enumerated and categorical histograms have 100 buckets. r=chutten,frank

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ff646e7183a3
Fix lint issue on test_histogramtools_strict.py a=lint-fix CLOSED TREE

Comment 4

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/3eb9521efab1
https://hg.mozilla.org/mozilla-central/rev/ff646e7183a3