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)

defect

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?
Flags: needinfo?(mreid)
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 =))
Depends on: 1314622
Merged in on 11/8
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Blocks: 1269466
Product: Cloud Services → Cloud Services Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: