Bug 1576573 Comment 2 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Summary of the discussion between @ssengupta, @sg and @janv today:
* Do not gather time measurements with telemetry for now. For measuring specific times, a different approach should be used, see Bug 1601214
* Data should be submitted to Telemetry when a cursor is closed:
   * The number of discarded entries for each Advance/Continue operation should be accumulated/summed up internally. (Only one sum for all of Advance/Continue/ContinuePrimaryKey).
   * In addition, the sum of invalidated entries should be submitted.
   * In addition, the number of entries accessed should be submitted, or the relation of discarded/invalidated entries to the number of entries accessed should be submitted.
* The same entry can only be discarded once, but it can be invalidated multiple times.
* Maybe use a single probe (keyed or categorical histogram) for all different metrics rather than multiple independent probes. Check with Telemetry team / #fxmetrics.
* The data should be used to see if we need to further optimize preloading (statically or dynamically), and in the future if additional optimizations are effective.
* Beyond the scope of this particular bug: This data could also be stored in the SQLite database (similar to maintenance statistics) for dynamic adjustments.
Summary of the discussion between @ssengupta, @sg and @janv today:
* Do not gather time measurements with telemetry for now. For measuring specific times, a different approach should be used, see Bug 1601214
* Data should be submitted to Telemetry when a cursor is closed:
   * The number of discarded entries for each Advance/Continue operation should be accumulated/summed up internally. (Only one sum for all of Advance/Continue/ContinuePrimaryKey).
   * In addition, the sum of invalidated entries should be submitted.
   * In addition, the number of entries accessed should be submitted, or the relation of discarded/invalidated entries to the number of entries accessed should be submitted.
   * We probably also need to know the value of the preloading preferences.
* The same entry can only be discarded once, but it can be invalidated multiple times.
* Maybe use a single probe (keyed or categorical histogram) for all different metrics rather than multiple independent probes. Check with Telemetry team / #fxmetrics.
* The data should be used to see if we need to further optimize preloading (statically or dynamically), and in the future if additional optimizations are effective.
* Beyond the scope of this particular bug: This data could also be stored in the SQLite database (similar to maintenance statistics) for dynamic adjustments.

Back to Bug 1576573 Comment 2