mlbiscoc commented on code in PR #3458: URL: https://github.com/apache/solr/pull/3458#discussion_r2263737412
########## solr/core/src/java/org/apache/solr/metrics/SolrCoreMetricManager.java: ########## @@ -52,6 +53,11 @@ public class SolrCoreMetricManager implements Closeable { private String leaderRegistryName; private boolean cloudMode; + // Track all metric producers registered for this core so we can re-initialize them during rename + private final List<MetricProducerInfo> registeredProducers = new ArrayList<>(); Review Comment: In Dropwizard, the "core registries" only needed to be renamed on the registries itself which worked because nothing had labels. Not possible in OTEL. We need to track our metric producers so we can re-register and reinitialize its metrics with the correct core name for labels. -- 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: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org