[Spotless] Using custom Java formatter from k-nn plugin

build.gradle

@@ -83,10 +83,10 @@ repositories {
 // Spotless checks will be added as PRs are applied to resolve each style issue is approved.
 spotless {
     java {
-//        target fileTree('.') {
-//            include '**/*.java', 'src/*/java/**/*.java'
-//            exclude '**/build/**', '**/build-*/**'
-//        }
+        target fileTree('.') {
+            include '**/*.java', 'src/*/java/**/*.java'
+            exclude '**/build/**', '**/build-*/**'
+        }
 //        importOrder()
 //        licenseHeader("/*\n" +
 //                " * Copyright OpenSearch Contributors\n" +
@@ -95,7 +95,8 @@ spotless {
 //        removeUnusedImports()
 //        trimTrailingWhitespace()
 //        endWithNewline()
-//        googleJavaFormat('1.17.0').reflowLongStrings().groupArtifact('com.google.googlejavaformat:google-java-format')
+        //googleJavaFormat('1.17.0').reflowLongStrings().groupArtifact('com.google.googlejavaformat:google-java-format')
+        eclipse().configFile rootProject.file('formatterConfig.xml')
     }
 }
 

core/src/main/java/org/opensearch/sql/analysis/AnalysisContext.java

@@ -3,7 +3,6 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-
 package org.opensearch.sql.analysis;
 
 import java.util.ArrayList;
@@ -17,56 +16,56 @@
  * The context used for Analyzer.
  */
 public class AnalysisContext {
-  /**
-   * Environment stack for symbol scope management.
-   */
-  private TypeEnvironment environment;
-  @Getter
-  private final List<NamedExpression> namedParseExpressions;
+    /**
+     * Environment stack for symbol scope management.
+     */
+    private TypeEnvironment environment;
+    @Getter
+    private final List<NamedExpression> namedParseExpressions;
 
-  @Getter
-  private final FunctionProperties functionProperties;
+    @Getter
+    private final FunctionProperties functionProperties;
 
-  public AnalysisContext() {
-    this(new TypeEnvironment(null));
-  }
+    public AnalysisContext() {
+        this(new TypeEnvironment(null));
+    }
 
-  /**
-   * Class CTOR.
-   * @param environment Env to set to a new instance.
-   */
-  public AnalysisContext(TypeEnvironment environment) {
-    this.environment = environment;
-    this.namedParseExpressions = new ArrayList<>();
-    this.functionProperties = new FunctionProperties();
-  }
+    /**
+     * Class CTOR.
+     * @param environment Env to set to a new instance.
+     */
+    public AnalysisContext(TypeEnvironment environment) {
+        this.environment = environment;
+        this.namedParseExpressions = new ArrayList<>();
+        this.functionProperties = new FunctionProperties();
+    }
 
-  /**
-   * Push a new environment.
-   */
-  public void push() {
-    environment = new TypeEnvironment(environment);
-  }
+    /**
+     * Push a new environment.
+     */
+    public void push() {
+        environment = new TypeEnvironment(environment);
+    }
 
-  /**
-   * Return current environment.
-   *
-   * @return current environment
-   */
-  public TypeEnvironment peek() {
-    return environment;
-  }
+    /**
+     * Return current environment.
+     *
+     * @return current environment
+     */
+    public TypeEnvironment peek() {
+        return environment;
+    }
 
-  /**
-   * Pop up current environment from environment chain.
-   *
-   * @return current environment (before pop)
-   */
-  public TypeEnvironment pop() {
-    Objects.requireNonNull(environment, "Fail to pop context due to no environment present");
+    /**
+     * Pop up current environment from environment chain.
+     *
+     * @return current environment (before pop)
+     */
+    public TypeEnvironment pop() {
+        Objects.requireNonNull(environment, "Fail to pop context due to no environment present");
 
-    TypeEnvironment curEnv = environment;
-    environment = curEnv.getParent();
-    return curEnv;
-  }
+        TypeEnvironment curEnv = environment;
+        environment = curEnv.getParent();
+        return curEnv;
+    }
 }