Skip to content
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

ESQL: Introduce language versioning #104890

Closed
6 tasks done
costin opened this issue Jan 30, 2024 · 2 comments
Closed
6 tasks done

ESQL: Introduce language versioning #104890

costin opened this issue Jan 30, 2024 · 2 comments
Assignees
Labels
:Analytics/ES|QL AKA ESQL >enhancement Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)

Comments

@costin
Copy link
Member

costin commented Jan 30, 2024

Description

As the ESQL grammar evolves, we need to establish some guidelines around the grammar and what are the expectations for clients using an old version or a a new version against a cluster that hasn't been fully upgraded.

Break-down of tasks

@elasticsearchmachine elasticsearchmachine added the needs:triage Requires assignment of a team area label label Jan 30, 2024
@costin costin removed the needs:triage Requires assignment of a team area label label Jan 31, 2024
@costin
Copy link
Member Author

costin commented Jan 31, 2024

Another aspect of versioning is dealing with different planning across nodes:

  1. a new node is upgraded and creates a different plan, that expects different local inputs
  2. some of the local nodes are not upgraded and for the same plan, create an old output
  3. the plan execution fails since the coordinator receives a different layout than the one expected from the old nodes.

To avoid this the coordinator would have to:

  • either use the new rule only if all nodes are upgraded
  • translate the old payload into a new one (assuming that is possible)

@elasticsearchmachine elasticsearchmachine added the needs:triage Requires assignment of a team area label label Jan 31, 2024
@kingherc kingherc added the :Analytics/ES|QL AKA ESQL label Feb 7, 2024
@elasticsearchmachine
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/ES|QL AKA ESQL >enhancement Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)
Projects
None yet
Development

No branches or pull requests

5 participants