wu-sheng commented on issue #9883: URL: https://github.com/apache/skywalking/issues/9883#issuecomment-1417973002
Why don't we change `PrometheusMetricConverter#escapedMetricsNameCache` directly to support `/` as well? I think `/` is illegal for MAL no matter the sources of metrics. And @hanahmily @kezhenxu94 We don't have Prometheus fetcher anymore, what do you suggest for a new name of `PrometheusMetricConverter`? This class name seems unreasonable in the codes. -- 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]
