1336865 - Add telemetry to measure time spent on constructing variable-length prefix set

Status: RESOLVED FIXED

(Toolkit :: Safe Browsing, defect, P2)

Description

[Dimi Lee [:dimi]]

Telemetry should at least include:
1.time spent constructing a VLPrefixSet
2.time spent allocating a VLPrefixSet
3.time spent loading PrefixSet from disk

Comment 1

[François Marier [:francois]]

(In reply to Dimi Lee[:dimi][:dlee] from comment #0)
> 2.time spent allocating a VLPrefixSet

https://telemetry.mozilla.org/new-pipeline/dist.html#!cumulative=0&end_date=2017-01-30&keys=__none__!__none__!__none__&max_channel_version=nightly%252F54&measure=URLCLASSIFIER_VLPS_FALLOCATE_TIME&min_channel_version=null&product=Firefox&sanitize=1&sort_keys=submissions&start_date=2017-01-23&table=0&trim=1&use_submission_date=0

> 3.time spent loading PrefixSet from disk

https://telemetry.mozilla.org/new-pipeline/dist.html#!cumulative=0&end_date=2017-01-30&keys=__none__!__none__!__none__&max_channel_version=nightly%252F54&measure=URLCLASSIFIER_VLPS_FILELOAD_TIME&min_channel_version=null&product=Firefox&sanitize=1&sort_keys=submissions&start_date=2017-01-23&table=0&trim=1&use_submission_date=0

Comment 2

[Dimi Lee [:dimi]]

Attachment: Bug 1336865 - Add telemetry to measure time spent on constructing variable-length prefix set.

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

Comment 3

[François Marier [:francois]]

Comment on attachment 8834297 [details]
Bug 1336865 - Add telemetry to measure time spent on constructing variable-length prefix set.

https://reviewboard.mozilla.org/r/110286/#review111438

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0505ef568bc7
Add telemetry to measure time spent on constructing variable-length prefix set. r=francois

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/0505ef568bc7