1430322 - Update telemetry expiry for predictor prefetch probes

Status: RESOLVED FIXED

(Core :: Networking, enhancement, P1)

Description

[u408661]

Specifically, PREDICTOR_PREFETCH_DECISION_REASON and PREDICTOR_PREFETCH_IGNORE_REASON are both set to expire in Firefox 60, but for now prefetch is disabled (unless people enable it), so they aren't particularly helpful until prefetch is enabled. I'm hoping that will happen sometime in the next 2 or 3 releases, so let's make those probes expire in 65 (just 'cause it's a nice round number) and hope things go better :)

Comment 1

[u408661]

Attachment: Bug 1430322 - Update expiration for PREDICTOR_PREFETCH_{DECISION,IGNORE}_REASON.

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

Comment 2

[Valentin Gosu [:valentin] (he/him)]

Comment on attachment 8942392 [details]
Bug 1430322 - Update expiration for PREDICTOR_PREFETCH_{DECISION,IGNORE}_REASON.

https://reviewboard.mozilla.org/r/212684/#review218384

Looks good to me. Make sure to check with the data steward if this needs data-review.

Comment 3

[u408661]

Rebecca - does this need data-review? (Updating expiry because, due to outside forces, we've been unable to enable the feature and collect the data we actually want.)

Comment 4

[[e-mail]]

No, it does not need additional data review.

Comment 5

[u408661]

(In reply to Rebecca Weiss from comment #4)
> No, it does not need additional data review.

Excellent, thanks!

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1080244fbe53
Update expiration for PREDICTOR_PREFETCH_{DECISION,IGNORE}_REASON. r=valentin

Comment 7

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/1080244fbe53