set default exporter to otlp

Author: SylvainJuge

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

@@ -28,10 +28,9 @@ public class PropertiesCustomizer implements Function<ConfigProperties, Map<Stri
   public Map<String, String> apply(ConfigProperties config) {
     Map<String, String> result = new HashMap<>();
 
-    // set default exporter to logging when not explicitly set
+    // set default exporter to 'otlp' to be consistent with SDK
     if (config.getList(METRICS_EXPORTER).isEmpty()) {
-      logger.info(METRICS_EXPORTER + " is not set, default of 'logging' will be used");
-      result.put(METRICS_EXPORTER, "logging");
+      result.put(METRICS_EXPORTER, "otlp");
     }
 
     // providing compatibility with the existing 'otel.jmx.interval.milliseconds' config option

jmx-scraper/src/test/java/io/opentelemetry/contrib/jmxscraper/config/PropertiesCustomizerTest.java

@@ -24,14 +24,14 @@ void tryGetConfigBeforeApply() {
   }
 
   @Test
-  void defaultLoggingExporter() {
+  void defaultOtlpExporter() {
     Map<String, String> map = new HashMap<>();
     map.put("otel.jmx.service.url", "dummy-url");
     map.put("otel.jmx.target.system", "jvm");
     ConfigProperties config = DefaultConfigProperties.createFromMap(map);
 
     PropertiesCustomizer customizer = new PropertiesCustomizer();
-    assertThat(customizer.apply(config)).containsEntry("otel.metrics.exporter", "logging");
+    assertThat(customizer.apply(config)).containsEntry("otel.metrics.exporter", "otlp");
   }
 
   @Test