Re: [PR] SOLR-17340: add cache on top of BeanInfo instrospector [solr]

2024-07-22 Thread via GitHub
dsmiley merged PR #2538: URL: https://github.com/apache/solr/pull/2538 -- 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.or

Re: [PR] SOLR-17340: add cache on top of BeanInfo instrospector [solr]

2024-07-22 Thread via GitHub
dsmiley commented on code in PR #2538: URL: https://github.com/apache/solr/pull/2538#discussion_r1686941492 ## solr/CHANGES.txt: ## @@ -152,6 +152,8 @@ Optimizations * SOLR-17349: SolrDocumentFetcher should always skip lazy field loading overhead if documentCache==null (Mich

Re: [PR] SOLR-17340: add cache on top of BeanInfo instrospector [solr]

2024-07-22 Thread via GitHub
psalagnac commented on code in PR #2538: URL: https://github.com/apache/solr/pull/2538#discussion_r1686889090 ## solr/core/src/java/org/apache/solr/util/stats/MetricUtils.java: ## @@ -827,7 +845,7 @@ public static void addMXBeanMetrics( try { final Class intf =