Repository navigation
Conversation
epugh
left a comment
There was a problem hiding this comment.
I love me a java class class called "Experimental" ;-). I wonder if we need the long comment... "Get CPU utlizatoin from JFR instead of JDG to work aroudn bugs"?
dsmiley
left a comment
There was a problem hiding this comment.
Just approving based on what you're saying. I no longer have the older JDK 21 to try it
|
Quick Review: |
https://issues.apache.org/jira/browse/SOLR-18544
Partially a regression from SOLR-18407 but also partially a JDK bug that got exposed from the OTEL upgrade.
What I found was running the
PlacementPluginIntegrationTest.testAttributeFetcherImplwith21.0.6-temfails but running with21.0.9-tempasses.This was because previously the metric was sourced from JFR. Then after the upgrade it started to source it from JMX which exposed from JDK bug. So this change just sources it from JFR again.