1375043 - Rename m(Keyed)Accumulations to m(Keyed)HistogramAccumulations in DiscardedData

Status: RESOLVED FIXED

(Toolkit :: Telemetry, enhancement, P4)

Description

[Chris H-C :chutten]

bug 1369041 introduces a new type called DiscardedData. It has a couple of members called mDiscardedAccumulations and mDiscardedKeyedAccumulations

According to bug 136904 comment 13 they should have the word "Histogram" in there someplace.

Comment 1

[Chris H-C :chutten]

Oops, that should be bug 1369041 comment 13.

Comment 2

[Krishna]

Hello,
I am new to open source. This could be my first bug. If no one is assigned, may I please work on it?

Comment 3

[Georg Fritzsche [:gfritzsche]]

Hello, welcome!

I'll assign the bug to you.
The first step is to get Firefox to build:
https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Build_Instructions/Simple_Firefox_build

Then you can do the renaming suggested here and build again: ./mach build
Once that works successfully, you can run our unit tests: ./mach test toolkit/components/telemetry/tests/unit/

With that working, you can upload a patch here and ask for review:
https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/How_to_Submit_a_Patch

The specific renaming here is to rename mDiscardedAccumulations to mDiscardedHistogramAccumulations & mDiscardedKeyedAccumulations to mDiscardedKeyedHistogramAccumulations:
https://dxr.mozilla.org/mozilla-central/search?q=regexp%3AmDiscardedKeyedAccumulations%7CmDiscardedAccumulations&redirect=false

There are more that we could rename [1], but we can take that to a separate patch or bug once the above is working.

1: https://dxr.mozilla.org/mozilla-central/search?q=regexp%3ADiscardedAccumulations%7CDiscardedKeyedAccumulations%7CAccumulation%7CKeyedAccumulation+path%3Atoolkit%2Fcomponents%2Ftelemetry&redirect=false

Comment 4

[Krishna]

Thank you for helping me out. :)
I ran the bootstrap python file and followed the prompts. But now I am stuck at 'Get the Mozilla Code' portion. I have installed Mercurial. But when I try to clone, it takes a long time(2 hrs) and doesn't complete. I guess it is because of the slow INTERNET connection. I figured I could use the bundles instead. So, do I have to download just the mozilla-central bundle or the others as well? Correct me if I am wrong.
I greatly appreciate your help.

Comment 5

[Georg Fritzsche [:gfritzsche]]

Right, our source code is a little big.
Just the mozilla-central bundle is enough to download.

If you use IRC [1], there are always people in the #introduction channel who can help with getting started.

1: https://wiki.mozilla.org/IRC

Comment 6

[Krishna]

Attachment: Patch for the bug 1375043(Rename m(Keyed)Accumulations to m(Keyed)HistogramAccumulations in DiscardedData)

Comment 7

[Krishna]

May I start working on [1], while the patch is reviewed? What do I have to rename?
1: https://dxr.mozilla.org/mozilla-central/search?q=regexp%3ADiscardedAccumulations%7CDiscardedKeyedAccumulations%7CAccumulation%7CKeyedAccumulation+path%3Atoolkit%2Fcomponents%2Ftelemetry&redirect=false

Comment 8

[Georg Fritzsche [:gfritzsche]]

Comment on attachment 8881024 [details] [diff] [review]
Patch for the bug 1375043(Rename m(Keyed)Accumulations to m(Keyed)HistogramAccumulations in DiscardedData)

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

Thanks, this looks good!

Comment 9

[Georg Fritzsche [:gfritzsche]]

I've tested that this builds fine locally, setting checkin-needed to get this landed.

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/751b38c592b5
Rename m(Keyed)Accumulations to m(Keyed)HistogramAccumulations in DiscardedData. r=gfritzsche

Comment 11

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/751b38c592b5