Skip to content

Commit 1e0d113

Browse files
committed
Exclude full-text search-filter operator tests on the analytics-engine route
The 7 comparison-operator tests in OperatorIT / CalciteOperatorIT use the search-filter syntax (source=idx age = 32), which lowers to a Lucene query_string predicate. Executing it on the analytics-engine route fails with a null LuceneReader: query_string needs an inverted-index searcher, which parquet-backed analytics indices don't have (DataFusion is not a full-text engine). Full-text search is genuinely unsupported there, so exclude these tests when tests.analytics.parquet_indices is set, following the established build.gradle exclusion pattern. Signed-off-by: Kai Huang <ahkcs@amazon.com>
1 parent 9a32644 commit 1e0d113

1 file changed

Lines changed: 14 additions & 0 deletions

File tree

integ-test/build.gradle

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1033,6 +1033,20 @@ task integTestRemote(type: RestIntegTestTask) {
10331033
// to the analytics-engine planner via index settings.
10341034
if (System.getProperty("tests.analytics.parquet_indices") != null) {
10351035
systemProperty 'tests.analytics.parquet_indices', System.getProperty("tests.analytics.parquet_indices")
1036+
1037+
// Full-text search is unsupported on the analytics-engine route: the search-filter syntax
1038+
// (`source=idx field=value`) lowers to a Lucene query_string, which needs an inverted-index
1039+
// reader that parquet-backed analytics indices don't have. Exclude the comparison-operator
1040+
// tests that rely on it (both OperatorIT and CalciteOperatorIT).
1041+
filter {
1042+
excludeTestsMatching '*OperatorIT.testEqualOperator'
1043+
excludeTestsMatching '*OperatorIT.testNotEqualOperator'
1044+
excludeTestsMatching '*OperatorIT.testLessOperator'
1045+
excludeTestsMatching '*OperatorIT.testLteOperator'
1046+
excludeTestsMatching '*OperatorIT.testGreaterOperator'
1047+
excludeTestsMatching '*OperatorIT.testGteOperator'
1048+
excludeTestsMatching '*OperatorIT.testNotOperator'
1049+
}
10361050
}
10371051

10381052
// Set default query size limit

0 commit comments

Comments
 (0)