Page MenuHomePhabricator

PerDbNameStatsdDataFactory will not submit its stats
Closed, ResolvedPublic

Description

Stats will not get emitted as MediaWiki::emitBufferedStatsdData is never called for the PerDbName factory.

Possible solutions:

  • When emitting events for the default StatsdDataFactory, also emit for the PerDbName factory
  • Instead create a wrapper class implementing StatsdDataFactoryInterface that takes the extra prefix to be added to metrics, resulting in a single buffering factory for stats to be emitted from, something like "PrefixingStatsdDataFactoryProxy"

The PerDbName factory was initially introduced in https://gerrit.wikimedia.org/r/#/c/443467/

Event Timeline

Change 453575 had a related patch set uploaded (by Addshore; owner: Addshore):
[mediawiki/core@master] Create and use PrefixingStatsdDataFactoryProxy

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

Change 453575 merged by jenkins-bot:
[mediawiki/core@master] Create and use PrefixingStatsdDataFactoryProxy in PerDbNameStatsdDataFactory

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

Addshore triaged this task as Medium priority.Aug 22 2018, 3:22 PM
Addshore moved this task from Active 🚁 to Closing ✔️ on the User-Addshore board.

This is now working :)