Fix spotless

Signed-off-by: Chen Dai <[email protected]>

Author: dai-chen

common/src/main/java/org/opensearch/sql/common/setting/Settings.java

@@ -7,12 +7,11 @@
 
 import com.google.common.base.Strings;
 import com.google.common.collect.ImmutableMap;
-import lombok.Getter;
-import lombok.RequiredArgsConstructor;
-
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
+import lombok.Getter;
+import lombok.RequiredArgsConstructor;
 
 /** Setting. */
 public abstract class Settings {

integ-test/src/test/java/org/opensearch/sql/legacy/ExplainIT.java

@@ -5,21 +5,20 @@
 
 package org.opensearch.sql.legacy;
 
+import static org.hamcrest.Matchers.containsString;
+import static org.hamcrest.Matchers.equalTo;
+import static org.opensearch.sql.legacy.TestsConstants.*;
+
 import com.google.common.io.Files;
+import java.io.File;
+import java.io.IOException;
+import java.nio.charset.StandardCharsets;
 import org.junit.Assert;
 import org.junit.Ignore;
 import org.junit.Test;
 import org.opensearch.client.Request;
 import org.opensearch.client.Response;
 
-import java.io.File;
-import java.io.IOException;
-import java.nio.charset.StandardCharsets;
-
-import static org.hamcrest.Matchers.containsString;
-import static org.hamcrest.Matchers.equalTo;
-import static org.opensearch.sql.legacy.TestsConstants.*;
-
 public class ExplainIT extends SQLIntegTestCase {
 
   @Override

integ-test/src/test/java/org/opensearch/sql/legacy/PluginIT.java

@@ -5,6 +5,12 @@
 
 package org.opensearch.sql.legacy;
 
+import static org.hamcrest.Matchers.equalTo;
+import static org.opensearch.sql.legacy.TestUtils.getResponseBody;
+import static org.opensearch.sql.legacy.TestsConstants.TEST_INDEX_ACCOUNT;
+
+import java.io.IOException;
+import java.util.Locale;
 import org.json.JSONObject;
 import org.junit.Test;
 import org.opensearch.client.Request;
@@ -13,13 +19,6 @@
 import org.opensearch.client.ResponseException;
 import org.opensearch.sql.plugin.rest.RestQuerySettingsAction;
 
-import java.io.IOException;
-import java.util.Locale;
-
-import static org.hamcrest.Matchers.equalTo;
-import static org.opensearch.sql.legacy.TestUtils.getResponseBody;
-import static org.opensearch.sql.legacy.TestsConstants.TEST_INDEX_ACCOUNT;
-
 public class PluginIT extends SQLIntegTestCase {
 
   @Override

legacy/src/main/java/org/opensearch/sql/legacy/query/OpenSearchActionFactory.java

@@ -5,6 +5,9 @@
 
 package org.opensearch.sql.legacy.query;
 
+import static org.opensearch.sql.legacy.domain.IndexStatement.StatementType;
+import static org.opensearch.sql.legacy.utils.Util.toSqlExpr;
+
 import com.alibaba.druid.sql.ast.SQLExpr;
 import com.alibaba.druid.sql.ast.expr.SQLAggregateExpr;
 import com.alibaba.druid.sql.ast.expr.SQLAllColumnExpr;
@@ -19,6 +22,9 @@
 import com.alibaba.druid.sql.dialect.mysql.visitor.MySqlASTVisitorAdapter;
 import com.alibaba.druid.sql.parser.SQLStatementParser;
 import com.google.common.annotations.VisibleForTesting;
+import java.sql.SQLFeatureNotSupportedException;
+import java.util.ArrayList;
+import java.util.List;
 import org.opensearch.client.Client;
 import org.opensearch.search.SearchHit;
 import org.opensearch.search.SearchHits;
@@ -49,13 +55,6 @@
 import org.opensearch.sql.legacy.rewriter.parent.SQLExprParentSetterRule;
 import org.opensearch.sql.legacy.rewriter.subquery.SubQueryRewriteRule;
 
-import java.sql.SQLFeatureNotSupportedException;
-import java.util.ArrayList;
-import java.util.List;
-
-import static org.opensearch.sql.legacy.domain.IndexStatement.StatementType;
-import static org.opensearch.sql.legacy.utils.Util.toSqlExpr;
-
 public class OpenSearchActionFactory {
 
   public static QueryAction create(Client client, String sql)

legacy/src/test/java/org/opensearch/sql/legacy/antlr/semantic/SemanticAnalyzerConfigTest.java

@@ -5,15 +5,15 @@
 
 package org.opensearch.sql.legacy.antlr.semantic;
 
+import static org.hamcrest.Matchers.*;
+
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.rules.ExpectedException;
 import org.opensearch.sql.legacy.antlr.OpenSearchLegacySqlAnalyzer;
 import org.opensearch.sql.legacy.antlr.SqlAnalysisConfig;
 import org.opensearch.sql.legacy.esdomain.LocalClusterState;
 
-import static org.hamcrest.Matchers.*;
-
 /** Test cases for semantic analysis configuration */
 public class SemanticAnalyzerConfigTest extends SemanticAnalyzerTestBase {
 

opensearch/src/main/java/org/opensearch/sql/opensearch/setting/OpenSearchSettings.java

@@ -5,9 +5,21 @@
 
 package org.opensearch.sql.opensearch.setting;
 
+import static org.opensearch.common.settings.Settings.EMPTY;
+import static org.opensearch.common.unit.TimeValue.timeValueDays;
+import static org.opensearch.common.unit.TimeValue.timeValueMinutes;
+import static org.opensearch.sql.common.setting.Settings.Key.ENCYRPTION_MASTER_KEY;
+
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableMap;
+import java.io.InputStream;
+import java.util.Collections;
+import java.util.List;
+import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
+import java.util.function.Consumer;
+import java.util.function.Function;
 import lombok.RequiredArgsConstructor;
 import lombok.extern.log4j.Log4j2;
 import org.opensearch.cluster.ClusterName;
@@ -20,19 +32,6 @@
 import org.opensearch.sql.common.setting.LegacySettings;
 import org.opensearch.sql.common.setting.Settings;
 
-import java.io.InputStream;
-import java.util.Collections;
-import java.util.List;
-import java.util.Map;
-import java.util.concurrent.ConcurrentHashMap;
-import java.util.function.Consumer;
-import java.util.function.Function;
-
-import static org.opensearch.common.settings.Settings.EMPTY;
-import static org.opensearch.common.unit.TimeValue.timeValueDays;
-import static org.opensearch.common.unit.TimeValue.timeValueMinutes;
-import static org.opensearch.sql.common.setting.Settings.Key.ENCYRPTION_MASTER_KEY;
-
 /** Setting implementation on OpenSearch. */
 @Log4j2
 public class OpenSearchSettings extends Settings {