Page MenuHomePhabricator

Implement StatsLib improvements
Closed, ResolvedPublic

Description

  1. Drop unused $labels parameter from CounterMetric->incrementBy()
  2. Allow CounterMetric->IncrementBy() to accept a float value
  3. Annotate TimingMetric->observe() with expected time unit (milliseconds)

Event Timeline

Change 965145 had a related patch set uploaded (by Cwhite; author: Cwhite):

[mediawiki/core@master] Stats: remove unused $label parameter

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

Change 965566 had a related patch set uploaded (by Cwhite; author: Cwhite):

[mediawiki/core@master] Stats: allow floats to be provided to CounterMetric->incrementBy()

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

Change 965567 had a related patch set uploaded (by Cwhite; author: Cwhite):

[mediawiki/core@master] Stats: update TimingMetric->observe() docstring

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

colewhite changed the task status from Open to In Progress.Oct 12 2023, 7:59 PM
colewhite claimed this task.

Change 965145 merged by jenkins-bot:

[mediawiki/core@master] Stats: remove unused $label parameter

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

Change 965566 merged by jenkins-bot:

[mediawiki/core@master] Stats: allow floats to be provided to CounterMetric->incrementBy()

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

Change 965567 merged by jenkins-bot:

[mediawiki/core@master] Stats: update TimingMetric->observe() docstring

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

colewhite added a subscriber: aaron.

Thanks, @aaron!