Closed Bug 1249073 Opened 8 years ago Closed 8 years ago

main-ping fields with undocumented structure

Categories

(Toolkit :: Telemetry, defect, P1)

defect
Points:
2

Tracking

()

RESOLVED FIXED
mozilla47
Tracking Status
firefox47 --- fixed

People

(Reporter: spenrose, Assigned: Dexter)

References

Details

(Whiteboard: [measurement:client])

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1243893 +++

The following fields could use documentation of their structure:

- payload/addonDetails
- payload/lateWrites
- payload/simpleMeasurements
- payload/slowSQL and slowSQLStartup
- payload/UIMeasurements
I can help a bit with some background on two of these:

slowSQL is a keyed histogram of SQL statements that are "slow". slowSQLStartup is the same thing, but only including statements that run during application startup.
(In reply to Chris H-C :chutten from comment #1)
> I can help a bit with some background on two of these:
> 
> slowSQL is a keyed histogram of SQL statements that are "slow".
> slowSQLStartup is the same thing, but only including statements that run
> during application startup.

Thanks! I have reverse-engineered a bit of it, but I'd like more definitive type information. Alessio, can you put this on your list?
Flags: needinfo?(alessio.placitelli)
Sure, I'll document the structure of those fields. It looks like the high level documentation for most of them is already there (slowSQL/slowSQLStartup as well). Leaving the ni? around as a reminder!
Priority: -- → P2
Priority: P2 → P1
Assignee: nobody → alessio.placitelli
Flags: needinfo?(alessio.placitelli)
Attached patch bug1249073.patchSplinter Review
This patch does not add any additional information to the simpleMeasurement's docs, as I'm not sure how to improve that.

It also fixes a typo in the webRTC docs ("releated" -> "related").
Attachment #8724726 - Flags: review?(gfritzsche)
Attachment #8724726 - Flags: review?(gfritzsche) → review+
Sam, do you have any suggestion about how to improve payload/simpleMeasurements? Anything that could be added on the current readthedocs - https://gecko.readthedocs.org/en/latest/toolkit/components/telemetry/telemetry/main-ping.html#simplemeasurements ?
Flags: needinfo?(spenrose)
I simply missed that section; apologies for the confusion. simpleMeasurements is well documented.

(In reply to Alessio Placitelli [:Dexter] from comment #5)
> Sam, do you have any suggestion about how to improve
> payload/simpleMeasurements? Anything that could be added on the current
> readthedocs -
> https://gecko.readthedocs.org/en/latest/toolkit/components/telemetry/
> telemetry/main-ping.html#simplemeasurements ?
Flags: needinfo?(spenrose)
No problem Sam, thanks for clarifying!

No try push needed for the check-in, as this only updates the docs.
Status: NEW → ASSIGNED
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/04634ec900b2
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: