1464447 - Should clarify that main process histograms are in main payload

Status: RESOLVED FIXED

(Toolkit :: Telemetry, enhancement)

Description

[William Lachance (:wlach)]

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 1

[William Lachance (:wlach)]

Attachment: Bug 1464447 - Clarify where parent process histograms/keyedHistograms are

Review commit: https://reviewboard.mozilla.org/r/246798/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/246798/

Comment 2

[Alessio Placitelli [:Dexter]]

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.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d3d46b1e864f
Clarify where parent process histograms/keyedHistograms are r=Dexter

Comment 4

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/d3d46b1e864f