Skip to content

[Lens] [ES|QL] Remove date fields from ES|QL SORT clause generation#251389

Merged
walterra merged 2 commits intoelastic:mainfrom
walterra:lens-esql-convert-fix-date-sort
Feb 4, 2026
Merged

[Lens] [ES|QL] Remove date fields from ES|QL SORT clause generation#251389
walterra merged 2 commits intoelastic:mainfrom
walterra:lens-esql-convert-fix-date-sort

Conversation

@walterra
Copy link
Copy Markdown
Contributor

@walterra walterra commented Feb 3, 2026

Summary

Part of https://github.com/elastic/kibana-team/issues/2736.

Updates ES|QL query generation in Lens to exclude date fields from the SORT clause when converting form-based layers to ES|QL.

Checklist

@walterra walterra self-assigned this Feb 3, 2026
@walterra walterra added Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// release_note:skip Skip the PR/issue when compiling release notes Feature:Lens backport:skip This PR does not require backporting v9.4.0 labels Feb 3, 2026
@walterra walterra marked this pull request as ready for review February 3, 2026 09:45
@walterra walterra requested a review from a team as a code owner February 3, 2026 09:45
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-visualizations (Team:Visualizations)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was checking the data table example and without the sort it does not look the same:

No sort

Image

With sort

It looks like the original

Image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Discussed offline: we'll keep it as is for now

@walterra walterra marked this pull request as draft February 3, 2026 13:12
@walterra walterra marked this pull request as ready for review February 4, 2026 15:59
Copy link
Copy Markdown
Contributor

@mariairiartef mariairiartef left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
lens 2.0MB 2.0MB -2.0B

cc @walterra

@walterra walterra merged commit 90b82c4 into elastic:main Feb 4, 2026
16 checks passed
@walterra walterra deleted the lens-esql-convert-fix-date-sort branch February 4, 2026 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting Feature:Lens release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants