Closed
Bug 1311174
Opened 8 years ago
Closed 8 years ago
Add PUSH_API_NOTIFY to main_summary
Categories
(Cloud Services Graveyard :: Metrics: Pipeline, defect, P2)
Cloud Services Graveyard
Metrics: Pipeline
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bugzilla, Assigned: bugzilla)
References
Details
Attachments
(1 file)
Per Bryan's comment here (https://bugzilla.mozilla.org/show_bug.cgi?id=1269466#c22) we should start using PUSH_API_NOTIFY to calculate push ER since it was changed to opt-out as of Firefox 50 (going out to release channel 11/8). I believe this also means we can remove PUSH_API_NOTIFICATION_RECEIVED from main_summary, since that's still opt-in.
:mreid, are you aware of anyone using PUSH_API_NOTIFICATION_RECEIVED?
|
||
Comment 1•8 years ago
|
||
That field was added in bug 1270482 - if the use cases there are obsoleted by this field, we should be safe to remove it.
I wonder if we can grep re:dash queries for that field...
Flags: needinfo?(mreid)
Verified that no queries were using PUSH_API_NOTIFICATION_RECEIVED (https://sql.telemetry.mozilla.org/queries/1504 -- the 1 count is from this query itself =))
|
||
Comment 3•8 years ago
|
||
Merged in on 11/8
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
|
||
Updated•6 years ago
|
Product: Cloud Services → Cloud Services Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•