Update spotless to meet JDK-21 baseline (#618) (#627)

(cherry picked from commit 6c4400d) Signed-off-by: Andriy Redko <andriy.redko@aiven.io> (cherry picked from commit f3acfe4)
opensearch-project · Feb 9, 2024 · 7d1817a · 7d1817a
1 parent a267e14
commit 7d1817a
Show file tree

Hide file tree

Showing 64 changed files with 32 additions and 60 deletions.
diff --git a/build.gradle b/build.gradle
@@ -6,11 +6,11 @@
 buildscript {
 
     ext {
-        opensearch_version = System.getProperty("opensearch.version", "2.12.0-SNAPSHOT")
+        opensearch_version = System.getProperty("opensearch.version", "2.13.0-SNAPSHOT")
         isSnapshot = "true" == System.getProperty("build.snapshot", "true")
         buildVersionQualifier = System.getProperty("build.version_qualifier", "")
 
-        default_bwc_version = System.getProperty("bwc.version", "2.10.0.0")
+        default_bwc_version = System.getProperty("bwc.version", "2.12.0.0")
         pa_bwc_version = System.getProperty("tests.bwc.version", default_bwc_version)
         baseName = "paBwcCluster"
 
@@ -48,9 +48,9 @@ buildscript {
 plugins {
     id 'java'
     id 'com.netflix.nebula.ospackage' version "11.6.0"
-    id 'com.github.spotbugs' version '5.0.13'
+    id 'com.github.spotbugs' version '6.0.7'
     id 'jacoco'
-    id 'com.diffplug.spotless' version '5.11.0'
+    id 'com.diffplug.spotless' version '6.25.0'
     id 'checkstyle'
     id 'org.ajoberstar.grgit' version '5.0.0'
     id 'org.gradle.test-retry' version '1.5.2'
@@ -64,7 +64,7 @@ checkstyle {
 
 spotbugsMain {
     excludeFilter = file("checkstyle/findbugs-exclude.xml")
-    effort = 'max'
+    effort = com.github.spotbugs.snom.Effort.MAX
     ignoreFailures = true
 
     reports {
@@ -104,7 +104,7 @@ noticeFile = rootProject.file('NOTICE.txt')
 spotless {
     java {
         licenseHeaderFile(file('license-header'))
-        googleJavaFormat('1.12.0').aosp()
+        googleJavaFormat('1.17.0').aosp()
         importOrder()
         removeUnusedImports()
         trimTrailingWhitespace()
@@ -212,7 +212,7 @@ tasks.withType(JavaCompile) {
 }
 
 jacoco {
-    toolVersion = "0.8.7"
+    toolVersion = "0.8.11"
 }
 
 jacocoTestReport {
@@ -295,6 +295,7 @@ dependencies {
             force "org.apache.commons:commons-lang3:${versions.commonslang}"
             force "org.slf4j:slf4j-api:2.0.0"
             force "org.apache.bcel:bcel:6.6.0" // This line should be removed once Spotbugs is upgraded to 4.7.4
+            force "com.google.guava:guava:${guavaVersion}"
         }
     }
 
@@ -668,7 +669,7 @@ task integTestRemote(type: RestIntegTestTask) {
     testClusters {
         "${baseName}$i" {
             testDistribution = "ARCHIVE"
-            versions = ["2.10.0-SNAPSHOT", opensearch_version]
+            versions = ["2.12.0-SNAPSHOT", opensearch_version]
             numberOfNodes = 3
             plugin(provider(new Callable<RegularFile>() {
                 @Override

diff --git a/licenses/netty-buffer-4.1.104.Final.jar.sha1 b/licenses/netty-buffer-4.1.104.Final.jar.sha1
diff --git a/licenses/netty-buffer-4.1.106.Final.jar.sha1 b/licenses/netty-buffer-4.1.106.Final.jar.sha1
@@ -0,0 +1 @@
+c5495ba59a627641b3a7c23f6bcb801874c7f7b0
diff --git a/licenses/netty-codec-4.1.104.Final.jar.sha1 b/licenses/netty-codec-4.1.104.Final.jar.sha1
diff --git a/licenses/netty-codec-4.1.106.Final.jar.sha1 b/licenses/netty-codec-4.1.106.Final.jar.sha1
@@ -0,0 +1 @@
+025171b63aa1e7a5fd8a7e4e660d6d3110241ea7
diff --git a/licenses/netty-codec-http-4.1.104.Final.jar.sha1 b/licenses/netty-codec-http-4.1.104.Final.jar.sha1
diff --git a/licenses/netty-codec-http-4.1.106.Final.jar.sha1 b/licenses/netty-codec-http-4.1.106.Final.jar.sha1
@@ -0,0 +1 @@
+21a07cdf0fc46b313fe2248f1275cdbdac0ba87b
diff --git a/licenses/netty-codec-http2-4.1.104.Final.jar.sha1 b/licenses/netty-codec-http2-4.1.104.Final.jar.sha1
diff --git a/licenses/netty-codec-http2-4.1.106.Final.jar.sha1 b/licenses/netty-codec-http2-4.1.106.Final.jar.sha1
@@ -0,0 +1 @@
+41515e8c51eeaaddceabdb4f86fbc5dbfc25b70e
diff --git a/licenses/netty-codec-socks-4.1.104.Final.jar.sha1 b/licenses/netty-codec-socks-4.1.104.Final.jar.sha1
diff --git a/licenses/netty-codec-socks-4.1.106.Final.jar.sha1 b/licenses/netty-codec-socks-4.1.106.Final.jar.sha1
@@ -0,0 +1 @@
+129f9bd6aa0cf28bfd7d45a8b6a598eed9c67702
diff --git a/licenses/netty-common-4.1.104.Final.jar.sha1 b/licenses/netty-common-4.1.104.Final.jar.sha1
diff --git a/licenses/netty-common-4.1.106.Final.jar.sha1 b/licenses/netty-common-4.1.106.Final.jar.sha1
@@ -0,0 +1 @@
+132defb4689f41b51b483b7202b22b6e89fe35fd
diff --git a/licenses/netty-handler-4.1.104.Final.jar.sha1 b/licenses/netty-handler-4.1.104.Final.jar.sha1
diff --git a/licenses/netty-handler-4.1.106.Final.jar.sha1 b/licenses/netty-handler-4.1.106.Final.jar.sha1
@@ -0,0 +1 @@
+874c970c4ff958b1140dde52bc17e6a9e7cde662
diff --git a/licenses/netty-handler-proxy-4.1.104.Final.jar.sha1 b/licenses/netty-handler-proxy-4.1.104.Final.jar.sha1
diff --git a/licenses/netty-handler-proxy-4.1.106.Final.jar.sha1 b/licenses/netty-handler-proxy-4.1.106.Final.jar.sha1
@@ -0,0 +1 @@
+bffc88b7c56d7b553453e4244247a1b0ba1fdc8a
diff --git a/licenses/netty-resolver-4.1.104.Final.jar.sha1 b/licenses/netty-resolver-4.1.104.Final.jar.sha1
diff --git a/licenses/netty-resolver-4.1.106.Final.jar.sha1 b/licenses/netty-resolver-4.1.106.Final.jar.sha1
@@ -0,0 +1 @@
+e185ae573db04939215f94d6ba869758dcecbde9
diff --git a/licenses/netty-transport-4.1.104.Final.jar.sha1 b/licenses/netty-transport-4.1.104.Final.jar.sha1
diff --git a/licenses/netty-transport-4.1.106.Final.jar.sha1 b/licenses/netty-transport-4.1.106.Final.jar.sha1
@@ -0,0 +1 @@
+8fcca44ae16b98e15965093e7696832019fd6f27
diff --git a/licenses/netty-transport-native-unix-common-4.1.104.Final.jar.sha1 b/licenses/netty-transport-native-unix-common-4.1.104.Final.jar.sha1
diff --git a/licenses/netty-transport-native-unix-common-4.1.106.Final.jar.sha1 b/licenses/netty-transport-native-unix-common-4.1.106.Final.jar.sha1
@@ -0,0 +1 @@
+2da179bd95903f0fa73218b8f0d02690c0cfbc94
diff --git a/licenses/performance-analyzer-commons-1.3.0-SNAPSHOT.jar.sha1 b/licenses/performance-analyzer-commons-1.3.0-SNAPSHOT.jar.sha1
@@ -1 +1 @@
-a7390b56028ff0ec41c0114036cec55a511bfe91
+4b687090e06c7e28c0728fb7138628b6139dbb4a
diff --git a/src/main/java/org/opensearch/performanceanalyzer/OpenSearchResources.java b/src/main/java/org/opensearch/performanceanalyzer/OpenSearchResources.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer;
 
-
 import org.opensearch.client.Client;
 import org.opensearch.cluster.service.ClusterService;
 import org.opensearch.common.settings.Settings;

diff --git a/src/main/java/org/opensearch/performanceanalyzer/action/PerformanceAnalyzerActionFilter.java b/src/main/java/org/opensearch/performanceanalyzer/action/PerformanceAnalyzerActionFilter.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.action;
 
-
 import java.util.concurrent.atomic.AtomicLong;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;

diff --git a/...ain/java/org/opensearch/performanceanalyzer/action/PerformanceAnalyzerActionListener.java b/...ain/java/org/opensearch/performanceanalyzer/action/PerformanceAnalyzerActionListener.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.action;
 
-
 import org.opensearch.OpenSearchException;
 import org.opensearch.action.bulk.BulkResponse;
 import org.opensearch.action.search.SearchResponse;

diff --git a/src/main/java/org/opensearch/performanceanalyzer/collectors/ValueCalculator.java b/src/main/java/org/opensearch/performanceanalyzer/collectors/ValueCalculator.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.collectors;
 
-
 import org.opensearch.action.admin.indices.stats.ShardStats;
 
 @FunctionalInterface

diff --git a/src/main/java/org/opensearch/performanceanalyzer/config/setting/ClusterSettingsManager.java b/src/main/java/org/opensearch/performanceanalyzer/config/setting/ClusterSettingsManager.java
@@ -88,6 +88,7 @@ public void addSubscriberForStringSetting(
             stringSettingListenerMap.put(setting, Collections.singletonList(listener));
         }
     }
+
     /** Bootstraps the listeners and tries to read initial values for cluster settings. */
     public void initialize() {
         if (!initialized) {
@@ -248,6 +249,7 @@ private void callStringSettingListeners(final Setting<String> setting, String se
             StatsCollector.instance().logException(OPENSEARCH_REQUEST_INTERCEPTOR_ERROR);
         }
     }
+
     /** Class that handles response to GET /_cluster/settings */
     private class ClusterSettingsResponseHandler implements ActionListener<ClusterStateResponse> {
         /**

diff --git a/...org/opensearch/performanceanalyzer/config/setting/PerformanceAnalyzerClusterSettings.java b/...org/opensearch/performanceanalyzer/config/setting/PerformanceAnalyzerClusterSettings.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.config.setting;
 
-
 import org.opensearch.common.settings.Setting;
 
 public final class PerformanceAnalyzerClusterSettings {

diff --git a/...arch/performanceanalyzer/config/setting/handler/ConfigOverridesClusterSettingHandler.java b/...arch/performanceanalyzer/config/setting/handler/ConfigOverridesClusterSettingHandler.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.config.setting.handler;
 
-
 import com.google.common.collect.ImmutableList;
 import java.io.IOException;
 import java.util.ArrayList;

diff --git a/...va/org/opensearch/performanceanalyzer/config/setting/handler/NodeStatsSettingHandler.java b/...va/org/opensearch/performanceanalyzer/config/setting/handler/NodeStatsSettingHandler.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.config.setting.handler;
 
-
 import org.opensearch.performanceanalyzer.config.PerformanceAnalyzerController;
 import org.opensearch.performanceanalyzer.config.setting.ClusterSettingListener;
 import org.opensearch.performanceanalyzer.config.setting.ClusterSettingsManager;

diff --git a/.../performanceanalyzer/config/setting/handler/PerformanceAnalyzerClusterSettingHandler.java b/.../performanceanalyzer/config/setting/handler/PerformanceAnalyzerClusterSettingHandler.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.config.setting.handler;
 
-
 import java.util.LinkedHashMap;
 import java.util.Map;
 import org.opensearch.performanceanalyzer.config.PerformanceAnalyzerController;

diff --git a/...rformanceanalyzer/http_action/config/PerformanceAnalyzerOverridesClusterConfigAction.java b/...rformanceanalyzer/http_action/config/PerformanceAnalyzerOverridesClusterConfigAction.java
@@ -78,7 +78,9 @@ public List<ReplacedRoute> replacedRoutes() {
         return REPLACED_ROUTES;
     }
 
-    /** @return the name of this handler. */
+    /**
+     * @return the name of this handler.
+     */
     @Override
     public String getName() {
         return PerformanceAnalyzerOverridesClusterConfigAction.class.getSimpleName();

diff --git a/...pensearch/performanceanalyzer/http_action/config/PerformanceAnalyzerResourceProvider.java b/...pensearch/performanceanalyzer/http_action/config/PerformanceAnalyzerResourceProvider.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.http_action.config;
 
-
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.collect.ImmutableSet;
 import java.io.BufferedReader;

diff --git a/...ain/java/org/opensearch/performanceanalyzer/http_action/whoami/TransportWhoAmIAction.java b/...ain/java/org/opensearch/performanceanalyzer/http_action/whoami/TransportWhoAmIAction.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.http_action.whoami;
 
-
 import org.opensearch.action.support.ActionFilters;
 import org.opensearch.action.support.HandledTransportAction;
 import org.opensearch.common.inject.Inject;

diff --git a/src/main/java/org/opensearch/performanceanalyzer/http_action/whoami/WhoAmIAction.java b/src/main/java/org/opensearch/performanceanalyzer/http_action/whoami/WhoAmIAction.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.http_action.whoami;
 
-
 import org.opensearch.action.ActionType;
 import org.opensearch.core.common.io.stream.Writeable;
 

diff --git a/src/main/java/org/opensearch/performanceanalyzer/http_action/whoami/WhoAmIRequest.java b/src/main/java/org/opensearch/performanceanalyzer/http_action/whoami/WhoAmIRequest.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.http_action.whoami;
 
-
 import org.opensearch.action.support.nodes.BaseNodesRequest;
 import org.opensearch.core.common.io.stream.StreamInput;
 

diff --git a/...main/java/org/opensearch/performanceanalyzer/http_action/whoami/WhoAmIRequestBuilder.java b/...main/java/org/opensearch/performanceanalyzer/http_action/whoami/WhoAmIRequestBuilder.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.http_action.whoami;
 
-
 import org.opensearch.action.ActionRequestBuilder;
 import org.opensearch.client.ClusterAdminClient;
 import org.opensearch.client.OpenSearchClient;

diff --git a/src/main/java/org/opensearch/performanceanalyzer/http_action/whoami/WhoAmIResponse.java b/src/main/java/org/opensearch/performanceanalyzer/http_action/whoami/WhoAmIResponse.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.http_action.whoami;
 
-
 import java.io.IOException;
 import org.opensearch.core.action.ActionResponse;
 import org.opensearch.core.common.io.stream.StreamOutput;

diff --git a/src/main/java/org/opensearch/performanceanalyzer/listener/SearchListener.java b/src/main/java/org/opensearch/performanceanalyzer/listener/SearchListener.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.listener;
 
-
 import org.opensearch.search.internal.SearchContext;
 
 interface SearchListener {

diff --git a/...ava/org/opensearch/performanceanalyzer/transport/PerformanceAnalyzerTransportChannel.java b/...ava/org/opensearch/performanceanalyzer/transport/PerformanceAnalyzerTransportChannel.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.transport;
 
-
 import java.io.IOException;
 import java.util.concurrent.atomic.AtomicLong;
 import org.apache.logging.log4j.LogManager;

diff --git a/...org/opensearch/performanceanalyzer/transport/PerformanceAnalyzerTransportInterceptor.java b/...org/opensearch/performanceanalyzer/transport/PerformanceAnalyzerTransportInterceptor.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.transport;
 
-
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.opensearch.performanceanalyzer.config.PerformanceAnalyzerController;

diff --git a/src/main/java/org/opensearch/performanceanalyzer/util/Utils.java b/src/main/java/org/opensearch/performanceanalyzer/util/Utils.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.util;
 
-
 import java.util.EnumSet;
 import java.util.HashMap;
 import java.util.Iterator;

diff --git a/src/test/java/org/opensearch/performanceanalyzer/AbstractTests.java b/src/test/java/org/opensearch/performanceanalyzer/AbstractTests.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer;
 
-
 import java.io.File;
 import org.apache.logging.log4j.Level;
 import org.apache.logging.log4j.core.Filter;

diff --git a/src/test/java/org/opensearch/performanceanalyzer/ConfigOverridesIT.java b/src/test/java/org/opensearch/performanceanalyzer/ConfigOverridesIT.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer;
 
-
 import com.google.gson.JsonObject;
 import com.google.gson.JsonParser;
 import java.io.InputStreamReader;

diff --git a/src/test/java/org/opensearch/performanceanalyzer/CustomMetricsLocationTestBase.java b/src/test/java/org/opensearch/performanceanalyzer/CustomMetricsLocationTestBase.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer;
 
-
 import java.io.File;
 import java.nio.file.Files;
 import java.nio.file.Path;

diff --git a/src/test/java/org/opensearch/performanceanalyzer/PerformanceAnalyzerIntegTestBase.java b/src/test/java/org/opensearch/performanceanalyzer/PerformanceAnalyzerIntegTestBase.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer;
 
-
 import com.fasterxml.jackson.core.type.TypeReference;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import java.io.IOException;

diff --git a/src/test/java/org/opensearch/performanceanalyzer/PerformanceAnalyzerRCAHealthCheckIT.java b/src/test/java/org/opensearch/performanceanalyzer/PerformanceAnalyzerRCAHealthCheckIT.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer;
 
-
 import com.fasterxml.jackson.databind.JsonNode;
 import java.util.Iterator;
 import java.util.Objects;

diff --git a/src/test/java/org/opensearch/performanceanalyzer/bwc/PABackwardsCompatibilityIT.java b/src/test/java/org/opensearch/performanceanalyzer/bwc/PABackwardsCompatibilityIT.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.bwc;
 
-
 import java.util.List;
 import java.util.Map;
 import java.util.Set;

diff --git a/.../java/org/opensearch/performanceanalyzer/collectors/CacheConfigMetricsCollectorTests.java b/.../java/org/opensearch/performanceanalyzer/collectors/CacheConfigMetricsCollectorTests.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.collectors;
 
-
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.module.paranamer.ParanamerModule;
 import java.io.IOException;

diff --git a/...test/java/org/opensearch/performanceanalyzer/collectors/CircuitBreakerCollectorTests.java b/...test/java/org/opensearch/performanceanalyzer/collectors/CircuitBreakerCollectorTests.java
@@ -5,7 +5,6 @@
 
 package org.opensearch.performanceanalyzer.collectors;
 
-
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.module.paranamer.ParanamerModule;
 import java.io.IOException;

diff --git a/...g/opensearch/performanceanalyzer/collectors/ClusterApplierServiceStatsCollectorTests.java b/...g/opensearch/performanceanalyzer/collectors/ClusterApplierServiceStatsCollectorTests.java
@@ -73,7 +73,9 @@ public void testClusterApplierServiceStats_saveMetricValues() {
     @Ignore
     @Test
     public void testClusterApplierServiceStats_collectMetrics()
-            throws NoSuchMethodException, IllegalAccessException, InvocationTargetException,
+            throws NoSuchMethodException,
+                    IllegalAccessException,
+                    InvocationTargetException,
                     JsonProcessingException {
         System.out.println("test 1");
         MetricsConfiguration.CONFIG_MAP.put(
@@ -128,7 +130,9 @@ public void testClusterApplierServiceStats_collectMetrics()
     @SuppressWarnings("unchecked")
     @Test
     public void testClusterApplierServiceStats_collectMetricsWithPreviousClusterApplierMetrics()
-            throws NoSuchMethodException, IllegalAccessException, InvocationTargetException,
+            throws NoSuchMethodException,
+                    IllegalAccessException,
+                    InvocationTargetException,
                     JsonProcessingException {
         System.out.println("test 2");
         MetricsConfiguration.CONFIG_MAP.put(