Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

ioanatia
Copy link
Contributor

@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
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
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
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

( ENRICH idx2 on f1 with f2 = f3 )
( 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
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
Contributor 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
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.

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:Search - Relevance The Search organization Search Relevance team test-release Trigger CI checks against release build v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants