Stats: convert TimingMetric usage of microtime() to hrtime()

Per PHP docs, hrtime() is recommended for performance measurements. Bug: T245464 Change-Id: I233f18faaeaccf70360e828383442abf036a1997
wikimedia · Oct 12, 2023 · 74cf32f · 74cf32f
1 parent 236fa50
commit 74cf32f
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/includes/libs/Stats/Metrics/TimingMetric.php b/includes/libs/Stats/Metrics/TimingMetric.php
@@ -70,7 +70,7 @@ public function __construct( $baseMetric, $logger ) {
  * @return void
  */
  public function start(): void {
- $this->startTime = microtime( true );
+ $this->startTime = hrtime( true );
  }
 
  /**
@@ -83,7 +83,8 @@ public function stop(): void {
  trigger_error( "Stats: stop() called before start() for metric '{$this->getName()}'", E_USER_WARNING );
  return;
  }
- $this->observe( ( microtime( true ) - $this->startTime ) * 1000 );
+ $value = ( hrtime( true ) - $this->startTime ) * 1e+6; // convert nanoseconds to milliseconds
+ $this->observe( $value );
  $this->startTime = null;
  }