Skip to content

QL: extract optimizer rules tests from SQL into QL#52169

Merged
astefan merged 1 commit intoelastic:masterfrom
astefan:51928_impl_tests_
Feb 12, 2020
Merged

QL: extract optimizer rules tests from SQL into QL#52169
astefan merged 1 commit intoelastic:masterfrom
astefan:51928_impl_tests_

Conversation

@astefan
Copy link
Copy Markdown
Contributor

@astefan astefan commented Feb 10, 2020

This is the second part of #51928

@elasticmachine
Copy link
Copy Markdown
Collaborator

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

@astefan astefan requested review from bpintea, costin and matriv February 11, 2020 00:04
@astefan astefan marked this pull request as ready for review February 11, 2020 00:04
Copy link
Copy Markdown
Contributor

@matriv matriv left a comment

Choose a reason for hiding this comment

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

LGTM

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 e5ad72b into elastic:master Feb 12, 2020
@astefan astefan deleted the 51928_impl_tests_ 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants