-
Notifications
You must be signed in to change notification settings - Fork 101
[DOCS] Add link to stored fields example #4576
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
base: main
Are you sure you want to change the base?
Conversation
Following you can find the validation results for the API you have changed.
You can validate this API yourself by using the |
Following you can find the validation results for the API you have changed.
You can validate this API yourself by using the |
Following you can find the validation results for the API you have changed.
You can validate this API yourself by using the |
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.
I left one suggestion.
* Object fields can't be returned;if specified, the request fails. | ||
* Only leaf fields can be retrieved with the `stored_fields` option. | ||
* Object fields can't be returned; if specified, the request fails. | ||
* For an example, refer to [Retrieve stored fields](https://www.elastic.co/docs/reference/elasticsearch/rest-apis/retrieve-stored-fields). |
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.
Sorry, I didn't mention it earlier, we can use ext_doc_id
here. Please add the next line to the table.csv file as part of this PR, so we can keep track the links we are adding:
retrieve-stored-fields,https://www.elastic.co/docs/reference/elasticsearch/rest-apis/retrieve-stored-fields
and then refer to the ID as the suggestions shows below, please.
* For an example, refer to [Retrieve stored fields](https://www.elastic.co/docs/reference/elasticsearch/rest-apis/retrieve-stored-fields). | |
* @ext_doc_id retrieve-stored-fields |
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.
I didn't think this was a good case for ext_doc_id because it's a section-specific thing that's not likely to be reused. Or is it required?
Related to elastic/docs-projects#302 and elastic/elasticsearch#129380
DO NOT MERGE BEFORE elastic/elasticsearch#129380
stored_fields
parameter description