[Asterisk-code-review] res_statsd: replace meter type with counter (asterisk[16])

Rijnhard Hessel asteriskteam at digium.com
Thu Jul 8 04:50:41 CDT 2021


Rijnhard Hessel has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/16149 )


Change subject: res_statsd: replace meter type with counter
......................................................................

res_statsd: replace meter type with counter

ASTERISK-25419

Change-Id: I5fcb385983a1b88f03696ff30a26b55c546a1dd7
---
M include/asterisk/statsd.h
M res/res_chan_stats.c
2 files changed, 2 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/49/16149/1

diff --git a/include/asterisk/statsd.h b/include/asterisk/statsd.h
index 4dbfb77..aa27c48 100644
--- a/include/asterisk/statsd.h
+++ b/include/asterisk/statsd.h
@@ -41,8 +41,7 @@
 #define AST_STATSD_TIMER "ms"
 /*! Distribution of values over time. */
 #define AST_STATSD_HISTOGRAM "h"
-/*! Events over time. Sorta like increment-only counters. */
-#define AST_STATSD_METER "m"
+
 
 /*!
  * \brief Send a stat to the configured statsd server.
diff --git a/res/res_chan_stats.c b/res/res_chan_stats.c
index dbc79f0..446f930 100644
--- a/res/res_chan_stats.c
+++ b/res/res_chan_stats.c
@@ -73,7 +73,7 @@
 	if (metric) {
 		ast_str_set(&metric, 0, "stasis.message.%s",
 			stasis_message_type_name(stasis_message_type(message)));
-		ast_statsd_log(ast_str_buffer(metric), AST_STATSD_METER, 1);
+		ast_statsd_log(ast_str_buffer(metric), AST_STATSD_COUNTER, 1);
 	}
 }
 

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/16149
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: I5fcb385983a1b88f03696ff30a26b55c546a1dd7
Gerrit-Change-Number: 16149
Gerrit-PatchSet: 1
Gerrit-Owner: Rijnhard Hessel <rijnhard at teleforge.co.za>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210708/7ddc8197/attachment.html>


More information about the asterisk-code-review mailing list