compile pass

Signed-off-by: Peng Huo <penghuo@gmail.com>

Author: penghuo

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

@@ -497,7 +497,7 @@ public LogicalPlan visitWindow(Window node, AnalysisContext context) {
     TypeEnvironment curEnv = context.peek();
     LogicalWindow window = (LogicalWindow) child;
     curEnv.define(
-        new Symbol(Namespace.FIELD_NAME, window.getWindowFunction().getNameOrAlias()),
+        new Symbol(Namespace.FIELD_NAME, window.getWindowFunction().getName()),
         window.getWindowFunction().getDelegated().type());
 
     return child;

core/src/main/java/org/opensearch/sql/ast/dsl/AstDSL.java

@@ -523,8 +523,7 @@ public static Window window(
             new WindowFunction(
                 new Function(patternMethod.name().toLowerCase(Locale.ROOT), funArgs),
                 List.of(),
-                List.of()),
-            alias),
+                List.of())),
         input);
   }
 

integ-test/build.gradle

@@ -435,10 +435,10 @@ integTest {
     }
 
     dependsOn ':opensearch-sql-plugin:bundlePlugin'
-    if(getOSFamilyType() != "windows") {
-        dependsOn startPrometheus
-        finalizedBy stopPrometheus
-    }
+//    if(getOSFamilyType() != "windows") {
+//        dependsOn startPrometheus
+//        finalizedBy stopPrometheus
+//    }
 
     // enable calcite codegen in IT
     systemProperty 'calcite.debug', 'false'

integ-test/src/test/java/org/opensearch/sql/calcite/standalone/CalcitePPLIntegTestCase.java

@@ -110,7 +110,6 @@ private Settings defaultSettings() {
       private final Map<Key, Object> defaultSettings =
           new ImmutableMap.Builder<Key, Object>()
               .put(Key.QUERY_SIZE_LIMIT, 200)
-              .put(Key.SQL_PAGINATION_API_SEARCH_AFTER, true)
               .put(Key.FIELD_TYPE_TOLERANCE, true)
               .put(Key.CALCITE_ENGINE_ENABLED, true)
               .put(Key.CALCITE_FALLBACK_ALLOWED, false)
@@ -135,7 +134,6 @@ protected Settings enablePushdown() {
       private final Map<Key, Object> defaultSettings =
           new ImmutableMap.Builder<Key, Object>()
               .put(Key.QUERY_SIZE_LIMIT, 200)
-              .put(Key.SQL_PAGINATION_API_SEARCH_AFTER, true)
               .put(Key.FIELD_TYPE_TOLERANCE, true)
               .put(Key.CALCITE_ENGINE_ENABLED, true)
               .put(Key.CALCITE_FALLBACK_ALLOWED, false)

opensearch/src/main/java/org/opensearch/sql/opensearch/util/RestRequestUtil.java

@@ -7,7 +7,7 @@
 
 import lombok.NonNull;
 import lombok.experimental.UtilityClass;
-import org.opensearch.client.node.NodeClient;
+import org.opensearch.transport.client.node.NodeClient;
 import org.opensearch.rest.RestChannel;
 import org.opensearch.rest.RestRequest;
 

ppl/src/main/java/org/opensearch/sql/ppl/parser/AstBuilder.java

@@ -96,6 +96,8 @@ public class AstBuilder extends OpenSearchPPLParserBaseVisitor<UnresolvedPlan> {
    */
   private final String query;
 
+  private final Settings settings = null;
+
   public AstBuilder(String query) {
     this(query, null);
   }
@@ -430,8 +432,7 @@ public UnresolvedPlan visitPatternsCommand(OpenSearchPPLParser.PatternsCommandCo
                             .toLowerCase(Locale.ROOT),
                     unresolvedArguments),
                 List.of(), // ignore partition by list for now as we haven't seen such requirement
-                List.of()), // ignore sort by list for now as we haven't seen such requirement
-            alias.get()));
+                List.of())));
   }
 
   @Override