Merge PR #285: Update span-stacktrace module to follow style guide

Author: trask

span-stacktrace/src/main/java/io/opentelemetry/contrib/stacktrace/StackTraceAutoConfig.java

@@ -5,6 +5,9 @@
 
 package io.opentelemetry.contrib.stacktrace;
 
+import static java.util.logging.Level.FINE;
+import static java.util.logging.Level.SEVERE;
+
 import com.google.auto.service.AutoService;
 import io.opentelemetry.sdk.autoconfigure.spi.AutoConfigurationCustomizer;
 import io.opentelemetry.sdk.autoconfigure.spi.AutoConfigurationCustomizerProvider;
@@ -19,7 +22,7 @@
 import javax.annotation.Nullable;
 
 @AutoService(AutoConfigurationCustomizerProvider.class)
-public class StackTraceAutoConfig implements AutoConfigurationCustomizerProvider {
+public final class StackTraceAutoConfig implements AutoConfigurationCustomizerProvider {
 
   private static final Logger log = Logger.getLogger(StackTraceAutoConfig.class.getName());
 
@@ -50,7 +53,7 @@ static long getMinDuration(ConfigProperties properties) {
       log.fine("Stack traces capture is disabled");
     } else {
       log.log(
-          Level.FINE,
+          FINE,
           "Stack traces will be added to spans with a minimum duration of {0} nanos",
           minDuration);
     }
@@ -70,7 +73,7 @@ static Predicate<ReadableSpan> getFilterPredicate(ConfigProperties properties) {
 
     if (filter == null) {
       // if value is set, lack of filtering is likely an error and must be reported
-      Level disabledLogLevel = filterClass != null ? Level.SEVERE : Level.FINE;
+      Level disabledLogLevel = filterClass != null ? SEVERE : FINE;
       log.log(disabledLogLevel, "Span stacktrace filtering disabled");
       return span -> true;
     } else {

span-stacktrace/src/main/java/io/opentelemetry/contrib/stacktrace/StackTraceSpanProcessor.java

@@ -14,7 +14,7 @@
 import java.io.StringWriter;
 import java.util.function.Predicate;
 
-public class StackTraceSpanProcessor implements ExtendedSpanProcessor {
+public final class StackTraceSpanProcessor implements ExtendedSpanProcessor {
 
   private final long minSpanDurationNanos;
 

span-stacktrace/src/test/java/io/opentelemetry/contrib/stacktrace/StackTraceAutoConfigTest.java

@@ -15,7 +15,7 @@
 import java.util.function.Predicate;
 import org.junit.jupiter.api.Test;
 
-public class StackTraceAutoConfigTest {
+class StackTraceAutoConfigTest {
 
   @Test
   void defaultConfig() {