Closed Bug 1871084 Opened 4 months ago Closed 3 months ago

Add a timeout for when the categorization is recorded

Categories

(Firefox :: Search, task, P2)

task

Tracking

()

RESOLVED FIXED
123 Branch
Tracking Status
firefox123 --- fixed

People

(Reporter: jteow, Assigned: jteow)

References

Details

(Whiteboard: [sng])

Attachments

(1 file)

Currently, we record a categorization when the user has left the page (closed tab, closed the app, navigated to another page).

DS would like for us to add a timeout:

I think the preferred choice for DS would be in 96min-293min range.

Assignee: nobody → jteow
Attachment #9372879 - Attachment description: WIP: - Bug 1871084 - Add a timeout for categorization impressions - r?scunnane → Bug 1871084 - Add a timeout for categorization impressions - r?scunnane
Status: NEW → ASSIGNED
Pushed by jteow@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/35427b371b33
Add a timeout for categorization impressions - r=scunnane
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 123 Branch

For historical context, while the bug info contains a larger range of time to wait, after talking with the Performance team, it was suggested we wait for user idleness before recording the event.

Since idleness can be interrupted, I suggested to DS that we shorten the time to 60 minutes since a super long timeout like hours could potentially not be triggered due to a user returning their computer and interacting with their mouse/keyboard. Ultimately, the point of the timeout is to make sure we don't have a huge delay between the SERP load and the reporting.

Additionally, we also record the last time a SERP was categorized. When the user wakes their computer up, we check the current time against the last time recorded and if it exceeds 60 minutes, we record a categorization impression.

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: