Remove the use of a FactoryBean when auto-configuring VirtualThreadMe…

…trics See spring-projectsgh-43957 Signed-off-by: Dmytro Nosan <[email protected]> Signed-off-by: arefbehboudi <[email protected]>
arefbehboudi · Jan 29, 2025 · e26806c · e26806c
1 parent 59572ad
commit e26806c
Showing 1 changed file with 4 additions and 46 deletions.
diff --git a/...a/org/springframework/boot/actuate/autoconfigure/metrics/JvmMetricsAutoConfiguration.java b/...a/org/springframework/boot/actuate/autoconfigure/metrics/JvmMetricsAutoConfiguration.java
@@ -16,9 +16,8 @@
 
 package org.springframework.boot.actuate.autoconfigure.metrics;
 
-import java.io.Closeable;
-
 import io.micrometer.core.instrument.MeterRegistry;
+import io.micrometer.core.instrument.binder.MeterBinder;
 import io.micrometer.core.instrument.binder.jvm.ClassLoaderMetrics;
 import io.micrometer.core.instrument.binder.jvm.JvmCompilationMetrics;
 import io.micrometer.core.instrument.binder.jvm.JvmGcMetrics;
@@ -31,9 +30,6 @@
 import org.springframework.aot.hint.RuntimeHints;
 import org.springframework.aot.hint.RuntimeHintsRegistrar;
 import org.springframework.beans.BeanUtils;
-import org.springframework.beans.factory.BeanClassLoaderAware;
-import org.springframework.beans.factory.DisposableBean;
-import org.springframework.beans.factory.FactoryBean;
 import org.springframework.boot.autoconfigure.AutoConfiguration;
 import org.springframework.boot.autoconfigure.EnableAutoConfiguration;
 import org.springframework.boot.autoconfigure.condition.ConditionalOnBean;
@@ -103,47 +99,9 @@ public JvmCompilationMetrics jvmCompilationMetrics() {
 	@ConditionalOnClass(name = VIRTUAL_THREAD_METRICS_CLASS)
 	@ConditionalOnMissingBean(type = VIRTUAL_THREAD_METRICS_CLASS)
 	@ImportRuntimeHints(VirtualThreadMetricsRuntimeHintsRegistrar.class)
-	VirtualThreadMetricsFactoryBean virtualThreadMetrics() {
-		return new VirtualThreadMetricsFactoryBean();
-	}
-
-	static final class VirtualThreadMetricsFactoryBean
-			implements FactoryBean<Object>, BeanClassLoaderAware, DisposableBean {
-
-		private ClassLoader classLoader;
-
-		private Class<?> instanceType;
-
-		private Object instance;
-
-		@Override
-		public void setBeanClassLoader(ClassLoader classLoader) {
-			this.classLoader = classLoader;
-		}
-
-		@Override
-		public Object getObject() {
-			if (this.instance == null) {
-				this.instance = BeanUtils.instantiateClass(getObjectType());
-			}
-			return this.instance;
-		}
-
-		@Override
-		public Class<?> getObjectType() {
-			if (this.instanceType == null) {
-				this.instanceType = ClassUtils.resolveClassName(VIRTUAL_THREAD_METRICS_CLASS, this.classLoader);
-			}
-			return this.instanceType;
-		}
-
-		@Override
-		public void destroy() throws Exception {
-			if (this.instance instanceof Closeable closeable) {
-				closeable.close();
-			}
-		}
-
+	MeterBinder virtualThreadMetrics() throws ClassNotFoundException {
+		Class<?> clazz = ClassUtils.forName(VIRTUAL_THREAD_METRICS_CLASS, getClass().getClassLoader());
+		return (MeterBinder) BeanUtils.instantiateClass(clazz);
 	}
 
 	static final class VirtualThreadMetricsRuntimeHintsRegistrar implements RuntimeHintsRegistrar {