Skip to content

EQL: define the list of optimization rules for the optimizer#52172

Merged
astefan merged 2 commits intoelastic:masterfrom
astefan:51929_impl
Feb 12, 2020
Merged

EQL: define the list of optimization rules for the optimizer#52172
astefan merged 2 commits intoelastic:masterfrom
astefan:51929_impl

Conversation

@astefan
Copy link
Copy Markdown
Contributor

@astefan astefan commented Feb 10, 2020

Addresses #51929

@astefan astefan added the :Analytics/EQL EQL querying label Feb 11, 2020
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-search (:Search/EQL)

@astefan astefan requested a review from costin February 11, 2020 00:10
@astefan astefan marked this pull request as ready for review February 11, 2020 00:10
Copy link
Copy Markdown
Member

@costin costin left a comment

Choose a reason for hiding this comment

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

LGTM

@astefan astefan merged commit 1f90d8c into elastic:master Feb 12, 2020
@astefan astefan deleted the 51929_impl branch February 12, 2020 04:57
astefan added a commit to astefan/elasticsearch that referenced this pull request Feb 12, 2020
astefan added a commit that referenced this pull request Feb 12, 2020
* Extract common optimizer tests (#52169)

(cherry picked from commit e5ad72b)

* Hook in the optimizer rules (#52172)

(cherry picked from commit 1f90d8c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/EQL EQL querying

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants