Skip to content

ES|QL: Make fork available in release builds#129606

Merged
ioanatia merged 7 commits intoelastic:mainfrom
ioanatia:fork_out_of_snapshot
Jun 19, 2025
Merged

ES|QL: Make fork available in release builds#129606
ioanatia merged 7 commits intoelastic:mainfrom
ioanatia:fork_out_of_snapshot

Conversation

@ioanatia
Copy link
Copy Markdown
Member

@ioanatia ioanatia commented Jun 18, 2025

related: #121950

This takes FORK out of snapshot.
FORK will initially be available as a tech preview feature.
We want to make sure the grammar changes land this week, so that they are made available in Kibana too before FF.

@ioanatia ioanatia added release highlight :Analytics/ES|QL AKA ESQL ES|QL-ui Impacts ES|QL UI v9.1.0 Team:Search - Relevance The Search organization Search Relevance team labels Jun 18, 2025
@ioanatia ioanatia mentioned this pull request Jun 17, 2025
23 tasks
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Hi @ioanatia, I've created a changelog YAML for you. Note that since this PR is labelled release highlight, you need to update the changelog YAML to fill out the extended information sections.

@ioanatia ioanatia added the test-release Trigger CI checks against release build label Jun 18, 2025
highlight:
title: Release FORK in tech preview
body: |-
Fork is a foundational building block that allows multiple branches of execution.
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.

💪 This is a thorough changelog entry.

Copy link
Copy Markdown
Contributor

@ChrisHegarty ChrisHegarty left a comment

Choose a reason for hiding this comment

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

Awesome!!! LGTM

( FORK ( WHERE a:"baz" ) ( EVAL x = [ 1, 2, 3 ] ) )
( COMPLETION a = b WITH c )
( SAMPLE 0.99 )
( INLINESTATS x = MIN(a), y = MAX(b) WHERE d > 1000 )
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.

Should we add RERANK as well?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I didn't want to add it before we finalize the syntax change - figured we might need to change it anyway

Copy link
Copy Markdown
Contributor

@afoucret afoucret left a comment

Choose a reason for hiding this comment

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

Overall OK.
Few questions but nothing blocking.

Copy link
Copy Markdown
Member

@carlosdelest carlosdelest left a comment

Choose a reason for hiding this comment

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

Congrats on having FORK out of snapshot! 🎉

@ioanatia ioanatia removed the test-release Trigger CI checks against release build label Jun 19, 2025
@ioanatia ioanatia marked this pull request as ready for review June 19, 2025 07:52
@elasticsearchmachine elasticsearchmachine added Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) and removed Team:Search - Relevance The Search organization Search Relevance team labels Jun 19, 2025
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/kibana-esql (ES|QL-ui)

@ioanatia ioanatia merged commit 04231e9 into elastic:main Jun 19, 2025
27 checks passed
@ioanatia ioanatia deleted the fork_out_of_snapshot branch June 19, 2025 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL ES|QL-ui Impacts ES|QL UI >feature release highlight Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants