Re-visit previously disabled spotbugs patterns and enable them (#17560)

codestyle/spotbugs-exclude.xml

@@ -137,15 +137,9 @@
     <Bug pattern="SWL_SLEEP_WITH_LOCK_HELD"/>
     <Bug pattern="UL_UNRELEASED_LOCK_EXCEPTION_PATH"/>
     <Bug pattern="URF_UNREAD_FIELD"/>
-    <!-- The following patterns have been excluded as part of upgrading to Java 17 as there were 100s of occurrences.
-         We should revisit these later.  -->
     <Bug pattern="CT_CONSTRUCTOR_THROW"/>
     <Bug pattern="SING_SINGLETON_HAS_NONPRIVATE_CONSTRUCTOR"/>
-    <Bug pattern="DCN_NULLPOINTER_EXCEPTION"/>
     <Bug pattern="SING_SINGLETON_INDIRECTLY_IMPLEMENTS_CLONEABLE"/>
     <Bug pattern="MS_EXPOSE_REP"/>
-    <Bug pattern="PA_PUBLIC_PRIMITIVE_ATTRIBUTE"/>
-    <Bug pattern="EI_EXPOSE_STATIC_REP2"/>
-    <Bug pattern="SS_SHOULD_BE_STATIC"/>
     <Bug pattern="SING_SINGLETON_IMPLEMENTS_SERIALIZABLE"/>
 </FindBugsFilter>

extensions-contrib/kubernetes-overlord-extensions/src/main/java/org/apache/druid/k8s/overlord/KubernetesAndWorkerTaskRunnerConfig.java

@@ -31,8 +31,8 @@ import javax.annotation.Nullable;
 public class KubernetesAndWorkerTaskRunnerConfig
 {
 
-  private final String RUNNER_STRATEGY_TYPE = "runnerStrategy.type";
+  private static final String RUNNER_STRATEGY_TYPE = "runnerStrategy.type";
-  private final String RUNNER_STRATEGY_WORKER_TYPE = "runnerStrategy.workerType";
+  private static final String RUNNER_STRATEGY_WORKER_TYPE = "runnerStrategy.workerType";
   /**
    * Select which runner type a task would run on, options are k8s or worker.
    */

extensions-core/druid-basic-security/src/main/java/org/apache/druid/security/basic/authentication/db/updater/CoordinatorBasicAuthenticatorMetadataStorageUpdater.java

@@ -74,7 +74,7 @@ public class CoordinatorBasicAuthenticatorMetadataStorageUpdater implements Basi
   private final BasicAuthCommonCacheConfig commonCacheConfig;
   private final ObjectMapper objectMapper;
   private final BasicAuthenticatorCacheNotifier cacheNotifier;
-  private final int numRetries = 5;
+  private static final int NUM_RETRIES = 5;
 
   private final Map<String, BasicAuthenticatorUserMapBundle> cachedUserMaps;
   private final Set<String> authenticatorPrefixes;
@@ -294,7 +294,7 @@ public class CoordinatorBasicAuthenticatorMetadataStorageUpdater implements Basi
   private void createUserInternal(String prefix, String userName)
   {
     int attempts = 0;
-    while (attempts < numRetries) {
+    while (attempts < NUM_RETRIES) {
       if (createUserOnce(prefix, userName)) {
         return;
       } else {
@@ -308,7 +308,7 @@ public class CoordinatorBasicAuthenticatorMetadataStorageUpdater implements Basi
   private void deleteUserInternal(String prefix, String userName)
   {
     int attempts = 0;
-    while (attempts < numRetries) {
+    while (attempts < NUM_RETRIES) {
       if (deleteUserOnce(prefix, userName)) {
         return;
       } else {
@@ -349,7 +349,7 @@ public class CoordinatorBasicAuthenticatorMetadataStorageUpdater implements Basi
     }
 
     int attempts = 0;
-    while (attempts < numRetries) {
+    while (attempts < NUM_RETRIES) {
       if (setUserCredentialOnce(prefix, userName, credentials)) {
         return;
       } else {

extensions-core/druid-basic-security/src/main/java/org/apache/druid/security/basic/authorization/db/updater/CoordinatorBasicAuthorizerMetadataStorageUpdater.java

@@ -91,7 +91,7 @@ public class CoordinatorBasicAuthorizerMetadataStorageUpdater implements BasicAu
   private final BasicAuthorizerCacheNotifier cacheNotifier;
   private final BasicAuthCommonCacheConfig commonCacheConfig;
   private final ObjectMapper objectMapper;
-  private final int numRetries = 5;
+  private static final int numRetries = 5;
 
   private final Map<String, BasicAuthorizerUserMapBundle> cachedUserMaps;
   private final Map<String, BasicAuthorizerGroupMappingMapBundle> cachedGroupMappingMaps;

extensions-core/druid-pac4j/src/main/java/org/apache/druid/security/pac4j/OIDCConfig.java

@@ -28,7 +28,7 @@ import javax.annotation.Nullable;
 
 public class OIDCConfig
 {
-  private final String DEFAULT_SCOPE = "name";
+  private static final String DEFAULT_SCOPE = "name";
   @JsonProperty
   private final String clientID;
 

extensions-core/histogram/src/main/java/org/apache/druid/query/aggregation/histogram/ApproximateHistogram.java

@@ -41,8 +41,8 @@ public class ApproximateHistogram
   // max size of the histogram (number of bincount/position pairs)
   int size;
 
-  public float[] positions;
+  protected float[] positions;
-  public long[] bins;
+  protected long[] bins;
 
   // used bincount
   int binCount;

processing/src/main/java/org/apache/druid/query/aggregation/Histogram.java

@@ -27,11 +27,11 @@ import java.util.Arrays;
 
 public class Histogram
 {
-  public float[] breaks;
+  protected float[] breaks;
-  public long[] bins;
+  protected long[] bins;
-  public transient long count;
+  protected transient long count;
-  public float min;
+  protected float min;
-  public float max;
+  protected float max;
 
   public Histogram(float[] breaks)
   {

processing/src/main/java/org/apache/druid/query/extraction/BucketExtractionFn.java

@@ -81,7 +81,7 @@ public class BucketExtractionFn implements ExtractionFn
     try {
       return bucket(Double.parseDouble(value));
     }
-    catch (NumberFormatException | NullPointerException ex) {
+    catch (NumberFormatException ex) {
       return null;
     }
   }

processing/src/main/java/org/apache/druid/segment/incremental/SpatialDimensionRowTransformer.java

@@ -213,10 +213,14 @@ public class SpatialDimensionRowTransformer implements Function<InputRow, InputR
   @Nullable
   private static Float tryParseFloat(String val)
   {
+    if (val == null) {
+      return null;
+    }
+
     try {
       return Float.parseFloat(val);
     }
-    catch (NullPointerException | NumberFormatException e) {
+    catch (NumberFormatException e) {
       return null;
     }
   }

server/src/main/java/org/apache/druid/server/initialization/jetty/JettyServerModule.java

@@ -597,7 +597,7 @@ public class JettyServerModule extends JerseyServletModule
   }
 
   @VisibleForTesting
-  public static void setJettyServerThreadPool(QueuedThreadPool threadPool)
+  protected static void setJettyServerThreadPool(QueuedThreadPool threadPool)
   {
     jettyServerThreadPool = threadPool;
   }