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.
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. * 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.