Closed Bug 1464447 Opened 2 years ago Closed 2 years ago

Should clarify that main process histograms are in main payload

Categories

(Toolkit :: Telemetry, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: wlach, Assigned: wlach)

Details

Attachments

(1 file)

Right now we don't clarify that the main process histograms are in the main payload under the processes section, which is confusing.

.. code-block:: js

    "processes" : {
      // ... other processes ...
      "parent": {
        scalars: {...},
        keyedScalars: {...},
        events: {...},
      },
      "content": {
        scalars: {...},
        keyedScalars: {...},
        histograms: {...},
        keyedHistograms: {...},
        events: {...},
      },
      "gpu": {
        // ...
      }
    }
Comment on attachment 8980638 [details]
Bug 1464447 - Clarify where parent process histograms/keyedHistograms are

https://reviewboard.mozilla.org/r/246798/#review252928

Mh, reading the docs again I think that you're right: it's not immediately clear that these metrics are in the top level `payload.histograms`. Thanks for fixing that.
Attachment #8980638 - Flags: review?(alessio.placitelli) → review+
Pushed by wlachance@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d3d46b1e864f
Clarify where parent process histograms/keyedHistograms are r=Dexter
https://hg.mozilla.org/mozilla-central/rev/d3d46b1e864f
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.