Skip to content

Commit 8c07b0c

Browse files
committed
Add more tests to exclusion list due to SORT
1 parent cfc37f2 commit 8c07b0c

File tree

1 file changed

+13
-4
lines changed
  • x-pack/plugin/esql/qa/server/multi-clusters/src/javaRestTest/java/org/elasticsearch/xpack/esql/ccq

1 file changed

+13
-4
lines changed

x-pack/plugin/esql/qa/server/multi-clusters/src/javaRestTest/java/org/elasticsearch/xpack/esql/ccq/MultiClusterSpecIT.java

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -111,13 +111,19 @@ public MultiClusterSpecIT(
111111
}
112112

113113
// TODO: think how to handle this better
114-
public static final Set<String> LOOKUP_JOIN_AFTER_STATS_TESTS = Set.of(
114+
public static final Set<String> NO_REMOTE_LOOKUP_JOIN_TESTS = Set.of(
115+
// Lookup join after STATS is not supported in CCS yet
115116
"StatsAndLookupIPAndMessageFromIndex",
116117
"JoinMaskingRegex",
117118
"StatsAndLookupIPFromIndex",
118119
"StatsAndLookupMessageFromIndex",
119120
"MvJoinKeyOnTheLookupIndexAfterStats",
120-
"MvJoinKeyOnFromAfterStats"
121+
"MvJoinKeyOnFromAfterStats",
122+
// Lookup join after SORT is not supported in CCS yet
123+
"SortBeforeAndAfterJoin",
124+
"SortEvalBeforeLookup",
125+
"NullifiedJoinKeyToPurgeTheJoin",
126+
"SortBeforeAndAfterMultipleJoinAndMvExpand"
121127
);
122128

123129
@Override
@@ -149,9 +155,12 @@ protected void shouldSkipTest(String testName) throws IOException {
149155
assumeFalse("FORK not yet supported in CCS", testCase.requiredCapabilities.contains(FORK_V9.capabilityName()));
150156
// Tests that use capabilities not supported in CCS
151157
assumeFalse(
152-
"LOOKUP JOIN after stats not yet supported in CCS",
153-
LOOKUP_JOIN_AFTER_STATS_TESTS.stream().anyMatch(testName::contains)
158+
"This syntax is not supported with remote LOOKUP JOIN",
159+
NO_REMOTE_LOOKUP_JOIN_TESTS.stream().anyMatch(testName::contains)
154160
);
161+
// Tests that do SORT before LOOKUP JOIN - not supported in CCS
162+
assumeFalse("LOOKUP JOIN after SORT not yet supported in CCS", testName.contains("OnTheCoordinator"));
163+
155164
}
156165

157166
@Override

0 commit comments

Comments
 (0)