Changes before error encountered

Co-authored-by: trask <[email protected]>

Author: Copilot

jmx-scraper/src/main/java/io/opentelemetry/contrib/jmxscraper/InvalidArgumentException.java

@@ -9,7 +9,7 @@
  * Exception indicating something is wrong with the provided arguments or reading the configuration
  * from them
  */
-public class InvalidArgumentException extends Exception {
+public final class InvalidArgumentException extends Exception {
 
   private static final long serialVersionUID = 0L;
 

jmx-scraper/src/main/java/io/opentelemetry/contrib/jmxscraper/JmxConnectorBuilder.java

@@ -5,6 +5,8 @@
 
 package io.opentelemetry.contrib.jmxscraper;
 
+import static java.util.logging.Level.WARNING;
+
 import com.google.errorprone.annotations.CanIgnoreReturnValue;
 import java.io.IOException;
 import java.net.MalformedURLException;
@@ -17,7 +19,6 @@
 import java.util.HashMap;
 import java.util.Locale;
 import java.util.Map;
-import java.util.logging.Level;
 import java.util.logging.Logger;
 import javax.annotation.Nullable;
 import javax.management.remote.JMXConnector;
@@ -33,7 +34,7 @@
 import javax.security.auth.callback.UnsupportedCallbackException;
 import javax.security.sasl.RealmCallback;
 
-public class JmxConnectorBuilder {
+public final class JmxConnectorBuilder {
 
   private static final Logger logger = Logger.getLogger(JmxConnectorBuilder.class.getName());
 
@@ -146,7 +147,7 @@ private Map<String, Object> buildEnv() {
                 }
               });
     } catch (ReflectiveOperationException e) {
-      logger.log(Level.WARNING, "SASL unsupported in current environment: " + e.getMessage());
+      logger.log(WARNING, "SASL unsupported in current environment: " + e.getMessage());
     }
     return env;
   }

jmx-scraper/src/main/java/io/opentelemetry/contrib/jmxscraper/JmxScraper.java

@@ -5,6 +5,10 @@
 
 package io.opentelemetry.contrib.jmxscraper;
 
+import static java.util.Collections.singletonList;
+import static java.util.logging.Level.INFO;
+import static java.util.logging.Level.SEVERE;
+
 import io.opentelemetry.api.GlobalOpenTelemetry;
 import io.opentelemetry.contrib.jmxscraper.config.JmxScraperConfig;
 import io.opentelemetry.contrib.jmxscraper.config.PropertiesCustomizer;
@@ -22,18 +26,16 @@
 import java.nio.file.Paths;
 import java.util.ArrayList;
 import java.util.Arrays;
-import java.util.Collections;
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
 import java.util.Properties;
 import java.util.concurrent.atomic.AtomicBoolean;
-import java.util.logging.Level;
 import java.util.logging.Logger;
 import javax.management.MBeanServerConnection;
 import javax.management.remote.JMXConnector;
 
-public class JmxScraper {
+public final class JmxScraper {
   private static final Logger logger = Logger.getLogger(JmxScraper.class.getName());
   private static final String CONFIG_ARG = "-config";
   private static final String TEST_ARG = "-test";
@@ -73,7 +75,7 @@ public static void main(String[] args) {
       JmxScraperConfig scraperConfig = configCustomizer.getScraperConfig();
 
       long exportSeconds = scraperConfig.getSamplingInterval().toMillis() / 1000;
-      logger.log(Level.INFO, "metrics export interval (seconds) =  " + exportSeconds);
+      logger.log(INFO, "metrics export interval (seconds) =  " + exportSeconds);
 
       JmxMetricInsight service =
           JmxMetricInsight.createService(
@@ -95,20 +97,20 @@ public static void main(String[] args) {
         jmxScraper.start();
       }
     } catch (ConfigurationException e) {
-      logger.log(Level.SEVERE, "invalid configuration: " + e.getMessage(), e);
+      logger.log(SEVERE, "invalid configuration: " + e.getMessage(), e);
       System.exit(1);
     } catch (InvalidArgumentException e) {
-      logger.log(Level.SEVERE, e.getMessage(), e);
+      logger.log(SEVERE, e.getMessage(), e);
       logger.info("Usage: java -jar <path_to_jmxscraper.jar> [-test] [-config <conf>]");
       logger.info("  -test           test JMX connection with provided configuration and exit");
       logger.info(
           "  -config <conf>  provide configuration, where <conf> is - for stdin, or <path_to_config.properties>");
       System.exit(1);
     } catch (IOException e) {
-      logger.log(Level.SEVERE, "Unable to connect ", e);
+      logger.log(SEVERE, "Unable to connect ", e);
       System.exit(2);
     } catch (RuntimeException e) {
-      logger.log(Level.SEVERE, e.getMessage(), e);
+      logger.log(SEVERE, e.getMessage(), e);
       System.exit(3);
     }
   }
@@ -119,14 +121,14 @@ private static boolean testConnection(JmxConnectorBuilder connectorBuilder) {
       MBeanServerConnection connection = connector.getMBeanServerConnection();
       Integer mbeanCount = connection.getMBeanCount();
       if (mbeanCount > 0) {
-        logger.log(Level.INFO, "JMX connection test OK");
+        logger.log(INFO, "JMX connection test OK");
         return true;
       } else {
-        logger.log(Level.SEVERE, "JMX connection test ERROR");
+        logger.log(SEVERE, "JMX connection test ERROR");
         return false;
       }
     } catch (IOException e) {
-      logger.log(Level.SEVERE, "JMX connection test ERROR", e);
+      logger.log(SEVERE, "JMX connection test ERROR", e);
       return false;
     }
   }
@@ -210,7 +212,7 @@ private void start() throws IOException {
 
     try (JMXConnector connector = client.build()) {
       MBeanServerConnection connection = connector.getMBeanServerConnection();
-      service.startRemote(getMetricConfig(config), () -> Collections.singletonList(connection));
+      service.startRemote(getMetricConfig(config), () -> singletonList(connection));
 
       running.set(true);
       logger.info("JMX scraping started");

jmx-scraper/src/main/java/io/opentelemetry/contrib/jmxscraper/config/JmxScraperConfig.java

@@ -5,12 +5,16 @@
 
 package io.opentelemetry.contrib.jmxscraper.config;
 
+import static java.util.Collections.emptyList;
+import static java.util.Collections.emptySet;
+import static java.util.Collections.unmodifiableList;
+import static java.util.Collections.unmodifiableSet;
+
 import io.opentelemetry.sdk.autoconfigure.spi.ConfigProperties;
 import io.opentelemetry.sdk.autoconfigure.spi.ConfigurationException;
 import java.io.InputStream;
 import java.time.Duration;
 import java.util.ArrayList;
-import java.util.Collections;
 import java.util.HashSet;
 import java.util.List;
 import java.util.Locale;
@@ -19,7 +23,7 @@
 import javax.annotation.Nullable;
 
 /** This class keeps application settings */
-public class JmxScraperConfig {
+public final class JmxScraperConfig {
 
   private static final Logger logger = Logger.getLogger(JmxScraperConfig.class.getName());
 
@@ -49,9 +53,9 @@ public class JmxScraperConfig {
 
   private String serviceUrl = "";
 
-  private List<String> jmxConfig = Collections.emptyList();
+  private List<String> jmxConfig = emptyList();
 
-  private Set<String> targetSystems = Collections.emptySet();
+  private Set<String> targetSystems = emptySet();
 
   private TargetSystemSource targetSystemSource = TargetSystemSource.AUTO;
 
@@ -217,8 +221,8 @@ public static JmxScraperConfig fromConfig(ConfigProperties config) {
           "at least one of '" + JMX_TARGET_SYSTEM + "' or '" + JMX_CONFIG + "' must be set");
     }
 
-    scraperConfig.jmxConfig = Collections.unmodifiableList(jmxConfig);
-    scraperConfig.targetSystems = Collections.unmodifiableSet(new HashSet<>(targetSystem));
+    scraperConfig.jmxConfig = unmodifiableList(jmxConfig);
+    scraperConfig.targetSystems = unmodifiableSet(new HashSet<>(targetSystem));
 
     scraperConfig.username = config.getString("otel.jmx.username");
     scraperConfig.password = config.getString("otel.jmx.password");

jmx-scraper/src/main/java/io/opentelemetry/contrib/jmxscraper/config/PropertiesCustomizer.java

@@ -16,7 +16,7 @@
 import javax.annotation.Nullable;
 
 /** Customizer of default SDK configuration and provider of effective scraper config */
-public class PropertiesCustomizer implements Function<ConfigProperties, Map<String, String>> {
+public final class PropertiesCustomizer implements Function<ConfigProperties, Map<String, String>> {
 
   private static final Logger logger = Logger.getLogger(PropertiesCustomizer.class.getName());
 

jmx-scraper/src/main/java/io/opentelemetry/contrib/jmxscraper/config/PropertiesSupplier.java

@@ -11,7 +11,7 @@
 import java.util.function.Supplier;
 
 /** Configuration supplier for java properties */
-public class PropertiesSupplier implements Supplier<Map<String, String>> {
+public final class PropertiesSupplier implements Supplier<Map<String, String>> {
 
   private final Properties properties;