Skip to content

Conversation

not-napoleon
Copy link
Member

Backports the following commits to 8.x:

…114056)

Resolves elastic#109998

For the most part, this is just adding tests. Greater and Least have actual production code changes - notably toEvaluator is modified to map date nanos to the long evaluator. This parallels the work done in elastic#113961. I've added CSV tests and unit tests for all the functions listed in the original ticket.


---------

Co-authored-by: Elastic Machine <[email protected]>
@not-napoleon not-napoleon added :Analytics/ES|QL AKA ESQL >non-issue 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) test-release Trigger CI checks against release build labels Oct 22, 2024
@not-napoleon
Copy link
Member Author

@elasticmachine run elasticsearch-ci/release-tests

@not-napoleon
Copy link
Member Author

@elasticmachine update branch

elasticmachine and others added 8 commits October 23, 2024 15:26
Cherry-pick
elastic@f8e931d#diff-41386766c394f14f5f205f92bb26eb1420b80af0057c78b2842fcc7ddd3d67aaR326

For whatever reason, git cherry-pick is having some difficulty with
this, so I just hand copied the mute.
 Conflicts:
	x-pack/plugin/esql/qa/testFixtures/src/main/resources/date_nanos.csv-spec
	x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/action/EsqlCapabilities.java
@elasticsearchmachine elasticsearchmachine merged commit 9b31ba2 into elastic:8.x Oct 24, 2024
16 checks passed
@not-napoleon not-napoleon deleted the backport/8.x/pr-114056 branch October 24, 2024 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >non-issue Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) test-release Trigger CI checks against release build v8.17.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants