Skip to content

[8.14] [ES|QL] Sorting accepts expressions (#181916)#182278

Merged
kibanamachine merged 1 commit intoelastic:8.14from
kibanamachine:backport/8.14/pr-181916
May 1, 2024
Merged

[8.14] [ES|QL] Sorting accepts expressions (#181916)#182278
kibanamachine merged 1 commit intoelastic:8.14from
kibanamachine:backport/8.14/pr-181916

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

## Summary

`SORT` accepts expressions as of
elastic/elasticsearch#107158.

This PR updates the validator and autocompletion engine to account for
this improvement.

### Checklist

- [ ]
[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added — looks like docs haven't been added for this feature yet. I
opened the discussion with ES team
([ref](elastic/elasticsearch#107158 (comment)))
- [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>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 5ba6a39)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label May 1, 2024
@kibanamachine kibanamachine enabled auto-merge (squash) May 1, 2024 17:37
@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 3.2MB 3.2MB +98.0B

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

cc @drewdaemon

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