Closed
Bug 1607315
Opened 6 years ago
Closed 6 years ago
Migrate session ping to the new data pipeline
Categories
(Firefox :: Messaging System, enhancement, P1)
Firefox
Messaging System
Tracking
()
Tracking | Status | |
---|---|---|
firefox74 | --- | verified |
People
(Reporter: nanj, Assigned: nanj)
References
Details
Attachments
(2 files)
To migrate this ping, we'd need to re-design its payload (a heavily nested structure) in order to work with the new data pipeline.
Also, various fields could be dropped as they are no longer of any interest.
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → najiang
Component: New Tab Page → Messaging System
Updated•6 years ago
|
Iteration: --- → 74.1 - Jan 6 - Jan 19
status-firefox73:
--- → wontfix
status-firefox74:
--- → affected
Priority: -- → P1
Assignee | ||
Updated•6 years ago
|
Iteration: 74.1 - Jan 6 - Jan 19 → ---
status-firefox73:
wontfix → ---
status-firefox74:
affected → ---
Depends on: dynamic-triplets
Priority: P1 → --
Comment 1•6 years ago
|
||
Assignee | ||
Comment 2•6 years ago
|
||
This schema was defined at here with a few minor changes compared to the old one:
- Remove the
region
field - Type change from "number" to "integer" for
load_trigger_ts
,visibility_event_rcvd_ts
, andtopsites_first_painted_ts
Pushed by najiang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6065b4a15418
Migrate Activity Stream session ping to the new data pipeline. r=andreio
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Blocks: dynamic-triplets, 1583272
No longer depends on: 1583272, dynamic-triplets
Comment 4•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 74
Comment 5•6 years ago
|
||
I have verified that the "load_trigger_ts", "visibility_event_rcvd_ts", and "topsites_first_painted_ts" values were added to the "Activity Stream" telemetry ping. Verified using latest Firefox Nightly (74.0a1 Build ID - 20200127093737) installed on Windows 10 x64, Mac 10.15.2, and Ubuntu 18.04 x64.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•