Skip to content

EQL: Backport to 7.x #51502

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 10 commits into from
Jan 28, 2020
Merged

EQL: Backport to 7.x #51502

merged 10 commits into from
Jan 28, 2020

Conversation

aleksmaus
Copy link
Contributor

@aleksmaus aleksmaus commented Jan 27, 2020 •
edited
Loading

The following commits were cherry picked from feature/eql branch for backport to 7.x

The extra commit to fix some incompatibilities between master and 7.x branch.
Please check that I didn't miss anything.
Thanks!

costin and others added 10 commits January 27, 2020 15:11
@costin @aleksmaus
Add draft EQL grammar and expression tree
10a16d1
@aleksmaus
Initial EQL rest API implementation (elastic#49768)
79875ce
@aleksmaus
Correct search_after handling (elastic#50629)
31d2d01
@imotov @aleksmaus
EQL: Make EqlSearchResponse immutable (elastic#50810)
6280831 
Refactors EqlSearchResponse to make it immutable

Relates to elastic#49581
@aleksmaus
Add more Eql REST API validation integration tests, clean up request …
d715176 
…implementation (elastic#50822)
@imotov @aleksmaus
EQL: Remove list classes from EqlSearchResponse (elastic#50870)
88cc30c 
Removes unnecessary classes from EqlSearchResponse that just represent
lists of other elements.

Relates to elastic#49581
@imotov @aleksmaus
EQL: Replace EqlSearchResponse.Hits parser with ObjectParser (elastic…
c184411 
…#50925)

Replaces the existing hand-build Hits parser with a
ConstructingObjectParser version.

Relates to elastic#49581
@costin @aleksmaus
EQL: import QL into EQL (elastic#50904)
d049de5 
Link QL into the new build file
Remove duplicate classes and use the new ql package
Update Exception hierarchy on top of QlException
@aleksmaus
Add xpack.eql.enabled feature flag, disabled by default. Enabled only…
d8f1735 
… for integration tests. (elastic#51370)

Related to elastic#49581
@aleksmaus
Compilation fixes for 7.x
eb1ed2a
@elasticmachine
Copy link
Collaborator

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

costin
costin approved these changes Jan 28, 2020
View reviewed changes
Copy link
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

@aleksmaus aleksmaus merged commit eb1ed2a into elastic:7.x Jan 28, 2020
@aleksmaus
Copy link
Contributor Author

Merged locally and pushed to 7.x to preserve history.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@costin costin costin approved these changes

@colings86 colings86 Awaiting requested review from colings86

@imotov imotov Awaiting requested review from imotov

@rw-access rw-access Awaiting requested review from rw-access

Assignees
No one assigned
Labels
:Analytics/EQL EQL querying backport
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@aleksmaus @elasticmachine @costin @imotov