-
Notifications
You must be signed in to change notification settings - Fork 25.1k
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] copy_to only works one level deep, not recursively #37249
Conversation
Pinging @elastic/es-search |
I also had an example snippet to demonstrate this scenario but thought that single bullet point describing it was enough:
Resulting in:
|
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.
@geekpete thanks, LGTM. Which version do you thing this addition should be targeted at? Do you want to merge or should I do that
6.x and 7.x versions of the docs I guess would be great. |
@geekpete thanks for merging, I also cherry-picked to 6.x and 6.6 since I think they were missed, or did you leave them out on purpose? |
Had just missed them not left out on purpose. |
Clarify that you cannot "chain" fields with
copy_to
and expect it to recursively copy via intermediary fields.