Merge pull request EverexIO#150 from timophey01/add-statsd-metrix

fix send-merics with server prefix
dustin1994 · May 30, 2019 · 36a98c0 · 36a98c0
2 parents b858283 + e099f70
commit 36a98c0
Showing 1 changed file with 14 additions and 0 deletions.
diff --git a/service/lib/metric.php b/service/lib/metric.php
@@ -97,6 +97,8 @@ static protected function getConnections() {
 
     static public function sendMetrics() {
         if (!(empty(self::$metric))) {
+            $time = microtime(true);
+            $endTime = false;
             self::sendAllTimingWithPrefix();
             if (
                 isset(self::$statsdOptions['duplicate_metrics_with_hostname']) &&
@@ -105,8 +107,20 @@ static public function sendMetrics() {
                 $hostname = explode('.', gethostname())[0];
                 if (!empty($hostname)) {
                     self::sendAllTimingWithPrefix($hostname);
+                    $endTime = microtime(true) - $time;
+                    self::$statsd->timing(
+                        $hostname . '.statsd.metric-send',
+                        $endTime
+                    );
                 }
             }
+            if (!$endTime) {
+                $endTime = microtime(true) - $time;
+            }
+            self::$statsd->timing(
+                self::STATSD_TOTAL_SERVER_NAME . '.statsd.metric-send',
+                $endTime
+            );
             self::$timings = [];
         }
     }