Closed Bug 1367750 Opened 4 years ago Closed 4 years ago

Allow new keys in Histograms.json if !strict_type_checks

Categories

(Toolkit :: Telemetry, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: chutten, Assigned: raajitr, Mentored)

References

Details

(Whiteboard: [lang=python])

Attachments

(1 file)

On the server side, histogram_tools doesn't need to be as strict as it is on client side. That's what strict type checks are for.

Let's expand this to allow new keys (like, say, the "keys" key from bug 1343855) so that we don't hamstring ourselves with requirements for tight synchronization of histogram_tools between client and server.
Mentor: gfritzsche, chutten
Priority: -- → P2
Whiteboard: [lang=python]
Attachment #8871445 - Flags: review?(alessio.placitelli) → review?(chutten)
Comment on attachment 8871445 [details]
Bug 1367750 - Allow new keys in Histograms.json if strict_type_checks is False

https://reviewboard.mozilla.org/r/142918/#review146950

Does this pass linting and building?
Attachment #8871445 - Flags: review?(chutten) → review+
Assignee: nobody → raajit.raj
(In reply to Chris H-C :chutten from comment #2)
> Comment on attachment 8871445 [details]
> Bug 1367750 - Allow new keys in Histograms.json if strict_type_checks is
> False
> 
> https://reviewboard.mozilla.org/r/142918/#review146950
> 
> Does this pass linting and building?
Flags: needinfo?(raajit.raj)
(In reply to Georg Fritzsche [:gfritzsche] from comment #3)
> (In reply to Chris H-C :chutten from comment #2)
> > Comment on attachment 8871445 [details]
> > Bug 1367750 - Allow new keys in Histograms.json if strict_type_checks is
> > False
> > 
> > https://reviewboard.mozilla.org/r/142918/#review146950
> > 
> > Does this pass linting and building?

Yes, it is.
Flags: needinfo?(raajit.raj)
Pushed by chutten@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f9acc145b45e
Allow new keys in Histograms.json if strict_type_checks is False r=chutten
https://hg.mozilla.org/mozilla-central/rev/f9acc145b45e
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.