Skip to content

[8.x] [ESQL] test date nanos union type (#116265)#116602

Merged
not-napoleon merged 5 commits intoelastic:8.xfrom
not-napoleon:backport/8.x/pr-116265
Nov 19, 2024
Merged

[8.x] [ESQL] test date nanos union type (#116265)#116602
not-napoleon merged 5 commits intoelastic:8.xfrom
not-napoleon:backport/8.x/pr-116265

Conversation

@not-napoleon
Copy link
Copy Markdown
Member

Backports the following commits to 8.x:

Resolves elastic#112885

This PR adds a bunch of basic testing for using TO_DATE_NANOS as a union type. It also tests the TO_DATETIME union type for casting date nanos. There are still some cases that aren't covered here, mostly because we haven't finished adding date nanos support to all the relevant functions. I expect we'll add in those cases as we add functions support.

---------

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@not-napoleon not-napoleon added :Analytics/ES|QL AKA ESQL >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Nov 11, 2024
@not-napoleon
Copy link
Copy Markdown
Member Author

@elasticmachine update branch

@not-napoleon not-napoleon removed the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Nov 19, 2024
@not-napoleon not-napoleon merged commit db57280 into elastic:8.x Nov 19, 2024
not-napoleon added a commit that referenced this pull request Nov 19, 2024
This should fix a couple of tests failures due to incorrect capabilities checks. I already added them manually to 8.x to resolve a failure there (see #116602) so this doesn't need to be backported.
rjernst pushed a commit to rjernst/elasticsearch that referenced this pull request Nov 20, 2024
This should fix a couple of tests failures due to incorrect capabilities checks. I already added them manually to 8.x to resolve a failure there (see elastic#116602) so this doesn't need to be backported.
alexey-ivanov-es pushed a commit to alexey-ivanov-es/elasticsearch that referenced this pull request Nov 28, 2024
This should fix a couple of tests failures due to incorrect capabilities checks. I already added them manually to 8.x to resolve a failure there (see elastic#116602) so this doesn't need to be backported.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.17.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants