Skip to content

[Lens][as-code] allow ES|QL datatables with no metrics#253856

Merged
drewdaemon merged 5 commits intoelastic:mainfrom
drewdaemon:252492/allow-datatable-with-no-metrics
Feb 20, 2026
Merged

[Lens][as-code] allow ES|QL datatables with no metrics#253856
drewdaemon merged 5 commits intoelastic:mainfrom
drewdaemon:252492/allow-datatable-with-no-metrics

Conversation

@drewdaemon
Copy link
Copy Markdown
Contributor

@drewdaemon drewdaemon commented Feb 18, 2026 •
edited
Loading

Summary

Fix #252492

Checklist

@drewdaemon drewdaemon changed the title [Lens] allow ES|QL datatables with no metrics [Lens][as-code] allow ES|QL datatables with no metrics Feb 18, 2026
@drewdaemon drewdaemon added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Feb 18, 2026
@drewdaemon
Copy link
Copy Markdown
Contributor Author

drewdaemon commented Feb 18, 2026

/ci

@drewdaemon drewdaemon force-pushed the 252492/allow-datatable-with-no-metrics branch from 22c617f to 46c4a49 Compare February 19, 2026 16:44
@drewdaemon drewdaemon marked this pull request as ready for review February 19, 2026 18:11
@drewdaemon drewdaemon requested a review from a team as a code owner February 19, 2026 18:11
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Feb 19, 2026

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #3 / Journey[web_logs_dashboard_long_running] Return to dashboard

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
agentBuilder 569.3KB 569.3KB -33.0B
dashboard 820.1KB 820.1KB -33.0B
dashboardAgent 173.2KB 173.2KB -33.0B
discover 1.6MB 1.6MB -33.0B
infra 1.2MB 1.2MB -33.0B
lens 2.0MB 2.0MB +12.0B
observability 2.0MB 2.0MB -33.0B
observabilityAIAssistantApp 287.3KB 287.3KB -33.0B
unifiedDocViewer 398.3KB 398.3KB -33.0B
total -252.0B

History

andrimal
andrimal reviewed Feb 20, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@andrimal andrimal left a comment

Choose a reason for hiding this comment

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

Tested locally and the issue is now fixed 🚀 I only left a couple of comments regarding validation.

Comment thread ...platform/packages/shared/kbn-lens-embeddable-utils/config_builder/schema/charts/datatable.ts
Comment thread ...hared/kbn-lens-embeddable-utils/config_builder/transforms/charts/datatable/to_api/columns.ts
@drewdaemon drewdaemon requested a review from andrimal February 20, 2026 16:04
@drewdaemon drewdaemon enabled auto-merge (squash) February 20, 2026 16:04
@drewdaemon drewdaemon merged commit 69f1590 into elastic:main Feb 20, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andrimal andrimal andrimal approved these changes

Assignees

No one assigned

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes v9.4.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Lens as code] Datatable metrics should be optional for ESQL panels

4 participants

@drewdaemon @elasticmachine @andrimal @kibanamachine