Page MenuHomePhabricator

Convert mw-js-deprecate from EventLogging to Statsv
Closed, ResolvedPublic

Description

This data seems like a waste, both for the client and for the server. Maybe it'd be more productive to reduce this to a simple statsv call.

This also has the side-effect of improving client-side overhead by doing debouncing and combining of multiple events, which is rather likely since deprecated call sites usually aren't lonely. Whereas right now, if a user is in the sample group, they create EventLogging beacons for every single deprecated function call as it happens.

We should convert statsd.js to use sendBeacon before we do this.

Event Timeline

Krinkle created this task.Dec 7 2015, 9:12 PM
Krinkle raised the priority of this task from to Needs Triage.
Krinkle updated the task description. (Show Details)
Krinkle added a subscriber: Krinkle.
Restricted Application added subscribers: StudiesWorld, Aklapper. · View Herald TranscriptDec 7 2015, 9:12 PM
Gilles triaged this task as Normal priority.Dec 14 2015, 7:51 PM
Gilles moved this task from Inbox to Backlog: Small & Maintenance on the Performance-Team board.
Gilles added a subscriber: Gilles.

Change 274622 had a related patch set uploaded (by Krinkle):
ext.wikimediaEvents.deprecate: Convert to use statsv instead of EventLogging

https://gerrit.wikimedia.org/r/274622

Krinkle claimed this task.Mar 3 2016, 2:49 AM

Change 274622 merged by jenkins-bot:
ext.wikimediaEvents.deprecate: Convert to use statsv instead of EventLogging

https://gerrit.wikimedia.org/r/274622