Skip to content

[8.14] [ES|QL] Fix validation on string implicit casting for dates and other minor issues (#181571)#181669

Merged
kibanamachine merged 1 commit intoelastic:8.14from
kibanamachine:backport/8.14/pr-181571
Apr 25, 2024
Merged

[8.14] [ES|QL] Fix validation on string implicit casting for dates and other minor issues (#181571)#181669
kibanamachine merged 1 commit intoelastic:8.14from
kibanamachine:backport/8.14/pr-181571

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.14:

Questions ?

Please refer to the Backport tool documentation

… minor issues (elastic#181571)

## Summary

Fixes validation for these scenarios where literal `string`s are
implicit casted to the other type by ES:

```
from index | where "2022" == now()
from index | where "2022" == 42
```

Also, it solves a bug that let two dates being added/diff each other,
like `eval now() - now()`.

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: Stratoula Kalafateli <efstratia.kalafateli@elastic.co>
(cherry picked from commit cf8a7fa)
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
kbnUiSharedDeps-srcJs 2.9MB 2.9MB +220.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @dej611

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants