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: Refactor top-level query planning #107953

Open
alex-spies opened this issue Apr 26, 2024 · 1 comment
Open

ESQL: Refactor top-level query planning #107953

alex-spies opened this issue Apr 26, 2024 · 1 comment
Assignees
Labels
:Analytics/ES|QL AKA ESQL >refactoring Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >tech debt

Comments

@alex-spies
Copy link
Contributor

ESQL's query planning currently uses multiple nested ActionListeners (see entrypoint here. Because of this, the code needs to be read last-to-first, starting with the plan optimization (last step), going deeper until the initial planning step (first step).

The ActionListeners seem to only contain procedural code, so this can probably be rewritten as simple procedural code: parse -> plan -> optimizations, resp. the number of action listener layers can likely be reduced.

@alex-spies alex-spies self-assigned this Apr 26, 2024
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Apr 26, 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 >refactoring Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >tech debt
Projects
None yet
Development

No branches or pull requests

2 participants