Skip to content

[ES|QL] Add FORK with KEEP/STATS in the transformational commands#240011

Merged
stratoula merged 1 commit intoelastic:mainfrom
stratoula:add-fork-transformational-commands
Oct 22, 2025
Merged

[ES|QL] Add FORK with KEEP/STATS in the transformational commands#240011
stratoula merged 1 commit intoelastic:mainfrom
stratoula:add-fork-transformational-commands

Conversation

@stratoula
Copy link
Contributor

Summary

Yesterday I was playing with FORK and discovered that in Discover we are trying to display the histogram if all branches are transformational while we shouldn't.

This PR updates the helper to take this scenario under consideration.

Before
image

After
image

Checklist

@stratoula stratoula added Feature:ES|QL ES|QL related features in Kibana Team:ESQL ES|QL related features in Kibana t// labels Oct 22, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Oct 22, 2025

🤖 Jobs for this PR can be triggered through checkboxes. 🚧

ℹ️ To trigger the CI, please tick the checkbox below 👇

  • Click to trigger kibana-pull-request for this PR!
  • Click to trigger kibana-deploy-project-from-pr for this PR!
  • Click to trigger kibana-deploy-cloud-from-pr for this PR!
@stratoula stratoula added v9.3.0 release_note:fix backport:skip This PR does not require backporting labels Oct 22, 2025
@stratoula stratoula marked this pull request as ready for review October 22, 2025 09:25
@stratoula stratoula requested a review from a team as a code owner October 22, 2025 09:25
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-esql (Team:ESQL)

Copy link
Contributor

@bartoval bartoval left a comment

Choose a reason for hiding this comment

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

only cr

@stratoula stratoula merged commit 181271b into elastic:main Oct 22, 2025
25 checks passed
NicholasPeretti pushed a commit to NicholasPeretti/kibana that referenced this pull request Oct 27, 2025
…astic#240011)

## Summary

Yesterday I was playing with FORK and discovered that in Discover we are
trying to display the histogram if all branches are transformational
while we shouldn't.

This PR updates the helper to take this scenario under consideration.

**Before**
<img width="2814" height="1066" alt="image"
src="https://github.com/user-attachments/assets/282ea349-6453-41cc-ae6c-fcd343b3d6c7"
/>


**After**
<img width="1514" height="723" alt="image"
src="https://github.com/user-attachments/assets/ec881ce4-84df-4799-9d80-3e05a8cc6122"
/>


### Checklist
- [ ] [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
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:ES|QL ES|QL related features in Kibana release_note:fix Team:ESQL ES|QL related features in Kibana t// v9.3.0

3 participants