-
Notifications
You must be signed in to change notification settings - Fork 25k
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
[DOCS] EQL: Document runs
keyword
#78478
Conversation
runs
keywordruns
keyword
Documents the `runs` keyword for repeating events in a sequence query. Relates to #75082.
Pinging @elastic/es-docs (Team:Docs) |
Pinging @elastic/es-ql (Team:QL) |
This statement is unclear since it suggests a matching event is going to be repeated in the response, which is not the case.
one can write
|
Thanks for the feedback @costin. I've pushed some changes to clarify the wording based on your feedback. Let me know if any further changes are needed. |
LGTM. The only remark is criteria is the plural for criterion so the first paragraph should read: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks @costin!
IMO the event category and event condition(s) are each a criterion. The "criteria" here is a set that consists of those components. It's certainly debatable tho. |
Documents the
runs
keyword for running the same event criteria successively in a sequence query.Relates to #75082.
Targeting 7.x to include the release highlight. I'll remove the highlight from the 8.0/master port.
Preview
runs
keyword docs: https://elasticsearch_78478.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/7.x/eql-syntax.html#eql-runs-keyword