Page MenuHomePhabricator

Review Popups metrics deprecation and coordinate according to WMDE Tech Wishes needs
Closed, InvalidPublic2 Estimated Story Points

Description

This patch intends to remove most of the Popups analytics, see T267211. We think this is great in the long-term, but some of the metrics are still important for us to establish baselines and monitor deployment of our Reference Previews and Cite-Extends features.

Review what will be removed, decide what we need and for how long, and coordinate with WMF.

Event Timeline

awight set the point value for this task to 2.Mar 3 2021, 9:29 AM
awight updated the task description. (Show Details)
awight updated the task description. (Show Details)

Thanks for writing this up, @awight.

Do note that the patch only intends to remove all instrumentation of the Popups schema and not the VirtualPageView and ReferencePreviewsPopups schemas.

@phuedx After squinting for a while, I see that we had at one point planned to piggy-back on the Popups schema, but never actually used the new fields and enum values. Everything we care about is in ReferencePreviewsPopups, as you pointed out!

I'll still follow your patch to verify that ReferencePreviewsPopups continues to be collected, but it looks good so far. Thank you for your patience :-)