Skip to content

Commit 89d1040

Browse files
authored
[analytics-engine] Strip AE-unsupported fields from test data and exclude the ITs doomed by it (opensearch-project#5541)
Signed-off-by: Eric Wei <mengwei.eric@gmail.com>
1 parent 7f31510 commit 89d1040

6 files changed

Lines changed: 930 additions & 3 deletions

File tree

integ-test/build.gradle

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1064,6 +1064,24 @@ task integTestRemote(type: RestIntegTestTask) {
10641064
excludeTestsMatching '*OperatorIT.testGreaterOperator'
10651065
excludeTestsMatching '*OperatorIT.testGteOperator'
10661066
excludeTestsMatching '*OperatorIT.testNotOperator'
1067+
1068+
// === Excludes: PPL ITs doomed by the unsupported-field strip ===
1069+
// On the AE route TestUtils.AnalyticsIndexConfig strips nested/geo_point/geo_shape/alias
1070+
// fields from every test mapping AND its bulk data (the parquet/composite store can't
1071+
// scan them). A test whose query references such a field can't pass — the data is gone.
1072+
// Verified field-by-field against the index mappings; only doomed work is excluded.
1073+
// Gated on the parsed boolean so `-Dtests.analytics.parquet_indices=false` excludes none.
1074+
//
1075+
// GeoPointFormatsIT: EVERY test reads a geo_point field — whole class doomed.
1076+
excludeTestsMatching 'org.opensearch.sql.ppl.GeoPointFormatsIT'
1077+
// datatypes_index_mapping: nested_value=nested, geo_point_value=geo_point.
1078+
excludeTestsMatching 'org.opensearch.sql.ppl.DataTypeIT.test_nonnumeric_data_types'
1079+
excludeTestsMatching 'org.opensearch.sql.ppl.SystemFunctionIT.typeof_opensearch_types'
1080+
// alias_index_mapping: alias_col is type=alias; query is `where alias_col > 1`.
1081+
excludeTestsMatching 'org.opensearch.sql.ppl.DataTypeIT.test_alias_data_type'
1082+
// CalciteAliasFieldAggregationIT: raw-PUT alias index can't be created on the AE route
1083+
// and every test queries alias fields directly — whole class doomed.
1084+
excludeTestsMatching 'org.opensearch.sql.calcite.remote.CalciteAliasFieldAggregationIT'
10671085
}
10681086
}
10691087

0 commit comments

Comments
 (0)