Skip to content

Commit

Permalink
Re-enable tests after warnings collection correction
Browse files Browse the repository at this point in the history
Re-enable a couple of warning header checking tests that have been
disabled, but mistakenly without linking the issue (elastic#100163), which lead
to them being kept disabled also after the fix (elastic#103031).
  • Loading branch information
bpintea committed Dec 18, 2023
1 parent 2f4f692 commit 6eb7c71
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -499,7 +499,7 @@ ROW deg = [90, 180, 270]
[90, 180, 270] | [1.5707963267948966, 3.141592653589793, 4.71238898038469]
;

warningWithFromSource-Ignore
warningWithFromSource
from employees | sort emp_no | limit 1 | eval x = to_long(emp_no) * 10000000 | eval y = to_int(x) > 1 | keep y;
warning:Line 1:89: evaluation of [to_int(x)] failed, treating result as null. Only first 20 failures recorded.
warning:Line 1:89: org.elasticsearch.xpack.ql.InvalidArgumentException: [100010000000] out of [integer] range
Expand All @@ -509,7 +509,7 @@ null
;

// the test is also notable through having the "failing" operation in the filter, which will be part of the fragment sent to a data node
multipleWarnings-Ignore
multipleWarnings
from employees | sort emp_no | eval x = to_long(emp_no) * 10000000 | where to_int(x) > 1 | keep x | limit 1;
warning:Line 1:76: evaluation of [to_int(x)] failed, treating result as null. Only first 20 failures recorded.
warning:Line 1:76: org.elasticsearch.xpack.ql.InvalidArgumentException: [100010000000] out of [integer] range
Expand Down

0 comments on commit 6eb7c71

Please sign in to comment.