[analytics-engine] Centralize CalciteAliasFieldAggregationIT exclude in build.gradle

Per review: keep all AE-route test exclusions in one place. Move the inline
Assume.assumeFalse(isAnalyticsParquetIndicesEnabled()) guard out of
CalciteAliasFieldAggregationIT into the gated analyticsEnabled exclude block in
integ-test/build.gradle, alongside the other doomed PPL ITs. The rationale
(raw-PUT alias index can't be created on the AE route) is carried into both the
build.gradle comment and a short note left at the test's init().

Signed-off-by: Eric Wei <mengwei.eric@gmail.com>

Author: mengweieric

integ-test/build.gradle

@@ -1079,6 +1079,9 @@ task integTestRemote(type: RestIntegTestTask) {
             excludeTestsMatching 'org.opensearch.sql.ppl.SystemFunctionIT.typeof_opensearch_types'
             // alias_index_mapping: alias_col is type=alias; query is `where alias_col > 1`.
             excludeTestsMatching 'org.opensearch.sql.ppl.DataTypeIT.test_alias_data_type'
+            // CalciteAliasFieldAggregationIT: raw-PUT alias index can't be created on the AE route
+            // and every test queries alias fields directly — whole class doomed.
+            excludeTestsMatching 'org.opensearch.sql.calcite.remote.CalciteAliasFieldAggregationIT'
         }
     }
 

integ-test/src/test/java/org/opensearch/sql/calcite/remote/CalciteAliasFieldAggregationIT.java

@@ -15,7 +15,6 @@
 import java.io.IOException;
 import java.util.List;
 import org.json.JSONObject;
-import org.junit.Assume;
 import org.junit.jupiter.api.Test;
 import org.opensearch.client.Request;
 import org.opensearch.client.ResponseException;
@@ -32,12 +31,9 @@ public class CalciteAliasFieldAggregationIT extends PPLIntegTestCase {
   @Override
   public void init() throws Exception {
     super.init();
-    // Alias fields are unsupported on the analytics-engine (parquet/composite) route — the index
-    // can't even be created, and these tests query the alias fields directly. There's nothing to
-    // salvage by stripping, so skip the whole class on the AE route. Runs normally otherwise.
-    Assume.assumeFalse(
-        "Alias-field aggregation is unsupported on the analytics-engine route",
-        isAnalyticsParquetIndicesEnabled());
+    // Excluded on the analytics-engine route from integ-test/build.gradle (alias fields are
+    // unsupported there — the raw-PUT index can't even be created and these tests query the alias
+    // fields directly), alongside the other AE-route exclusions kept in one place.
     enableCalcite();
     createTestIndexWithAliasFields();
     loadIndex(Index.DATA_TYPE_ALIAS);