-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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] Fixing field context examples #76887
[DOCS] Fixing field context examples #76887
Conversation
Pinging @elastic/es-core-infra (Team:Core/Infra) |
Pinging @elastic/es-docs (Team:Docs) |
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! Just one minor comment.
} | ||
}, | ||
"number-of-actors": { | ||
"script": { | ||
"source": "params['_source']['actors'].size()" | ||
"source": "doc['actors'].size()" |
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.
@stu-elastic Just wanted to confirm this is as expected.
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
Updates the examples for the Painless field context and also modifies the context examples so that all actors are lists.
Preview link: https://elasticsearch_76887.docs-preview.app.elstc.co/guide/en/elasticsearch/painless/master/painless-field-context.html
Resolves #76799