Closed Bug 1324003 Opened 6 years ago Closed 6 years ago

Remove the TELEMETRY_SCHEDULER_SEND_DAILY histogram

Categories

(Toolkit :: Telemetry, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: gfritzsche, Assigned: vincent, Mentored)

Details

(Whiteboard: [measurement:client] [lang=js] [good first bug])

Attachments

(1 file)

The TELEMETRY_SCHEDULER_SEND_DAILY expired and didn't yield any surprising result.
We should remove it and the accompanying code:
https://dxr.mozilla.org/mozilla-central/search?q=TELEMETRY_SCHEDULER_SEND_DAILY
Hi,

can I work on this? :-)
Please do!

You will need to remove the probe definition from toolkit/components/telemetry/Histograms.json and its use in toolkit/components/telemetry/TelemetrySession.jsm. Then you will need to build the code and test that the resulting firefox works by just seeing if it runs, and by running some tests.

Please let me know if you have any questions.
Assignee: nobody → sky
Mentor: chutten
Attachment #8820035 - Flags: review?(chutten)
Comment on attachment 8820035 [details] [diff] [review]
remove_telemetry_scheduler_send_daily.patch

Review of attachment 8820035 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM
Attachment #8820035 - Flags: review?(chutten) → review+
There, that should get it in the build in the next day or two.

Do you require help finding another bug to work on? We have plenty :)
Keywords: checkin-needed
Yes, thanks for that
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c37453397645
Remove TELEMETRY_SCHEDULER_SEND_DAILY; r=chutten
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/c37453397645
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.