empiredan commented on code in PR #1237:
URL:
https://github.com/apache/incubator-pegasus/pull/1237#discussion_r1022782659
##########
src/utils/metrics.h:
##########
@@ -331,6 +353,17 @@ class metric_prototype_with : public metric_prototype
DISALLOW_COPY_AND_ASSIGN(metric_prototype_with);
};
+const std::string kMetricTypeField = "type";
+const std::string kMetricNameField = "name";
+const std::string kMetricUnitField = "unit";
+const std::string kMetricDescField = "desc";
+const metric_fields_type kMetricPrototypeFields = {
Review Comment:
> Will kMetricPrototypeFields be used anywhere else except
get_all_single_value_metric_fields(), if not, make it there as a local variable
and do not expose it in header.
OK, I'll change these variables as local.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
