pr review

Author: zeitlinger

instrumentation/spring/spring-boot-autoconfigure/src/main/java/io/opentelemetry/instrumentation/spring/autoconfigure/internal/instrumentation/runtimemetrics/Java8RuntimeMetricsProvider.java

@@ -24,7 +24,7 @@ public class Java8RuntimeMetricsProvider implements RuntimeMetricsProvider {
 
   @Override
   public int minJavaVersion() {
-    return 8;
+    return 1;
   }
 
   @Override

instrumentation/spring/spring-boot-autoconfigure/src/main/java/io/opentelemetry/instrumentation/spring/autoconfigure/internal/instrumentation/runtimemetrics/RuntimeMetricsAutoConfiguration.java

@@ -48,7 +48,7 @@ public void handleApplicationReadyEvent(ApplicationReadyEvent event) {
     ConfigPropertiesBridge config =
         new ConfigPropertiesBridge(applicationContext.getBean(ConfigProperties.class));
 
-    double version = javaVersion();
+    double version = Double.parseDouble(System.getProperty("java.specification.version"));
     Optional<RuntimeMetricsProvider> metricsProvider =
         applicationContext.getBeanProvider(RuntimeMetricsProvider.class).stream()
             .sorted(Comparator.comparing(RuntimeMetricsProvider::minJavaVersion).reversed())
@@ -61,13 +61,4 @@ public void handleApplicationReadyEvent(ApplicationReadyEvent event) {
       logger.debug("No runtime metrics instrumentation available for Java {}", version);
     }
   }
-
-  private static int javaVersion() {
-    try {
-      return Integer.parseInt(System.getProperty("java.specification.version"));
-    } catch (NumberFormatException nfe) {
-      // The value of the java.specification.version property is 1.8 with a Java 8 runtime.
-      return 8;
-    }
-  }
 }