Skip to content

Commit

Permalink
Skip LOOKUP/INLINESTATS cases unless on snapshot (#111755) (#111759)
Browse files Browse the repository at this point in the history
Fix #111753

(cherry picked from commit f0d06ee)

# Conflicts:
#	muted-tests.yml
#	x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/analysis/VerifierTests.java
  • Loading branch information
alex-spies authored Aug 9, 2024
1 parent 6e26356 commit ba5ca2b
Showing 1 changed file with 12 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -181,16 +181,18 @@ public void testUnsupportedAndMultiTypedFields() {
error("from test* | stats values(multi_typed)", analyzer)
);

// LOOKUP with unsupported type
assertEquals(
"1:41: column type mismatch, table column was [integer] and original column was [unsupported]",
error("from test* | lookup int_number_names on int", analyzer)
);
// LOOKUP with multi-typed field
assertEquals(
"1:44: column type mismatch, table column was [double] and original column was [unsupported]",
error("from test* | lookup double_number_names on double", analyzer)
);
if (Build.current().isSnapshot()) {
// LOOKUP with unsupported type
assertEquals(
"1:41: column type mismatch, table column was [integer] and original column was [unsupported]",
error("from test* | lookup int_number_names on int", analyzer)
);
// LOOKUP with multi-typed field
assertEquals(
"1:44: column type mismatch, table column was [double] and original column was [unsupported]",
error("from test* | lookup double_number_names on double", analyzer)
);
}

assertEquals(
"1:24: Cannot use field [unsupported] with unsupported type [flattened]",
Expand Down

0 comments on commit ba5ca2b

Please sign in to comment.