Skip to content
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

Add support for missing value fetchers. #63578

Merged
merged 1 commit into from
Oct 12, 2020

Conversation

jtibshirani
Copy link
Contributor

Backport of #63515.

@jtibshirani jtibshirani added >enhancement :Search/Search Search-related issues that do not fall into other categories backport labels Oct 12, 2020
@elasticmachine
Copy link
Collaborator

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

@elasticmachine elasticmachine added the Team:Search Meta label for search team label Oct 12, 2020
This PR implements value fetching for the following field types:
* `text` phrase and prefix subfields
* `search_as_you_type`, plus its subfields
* `token_count`, which is implemented by fetching doc values

Supporting these types helps ensure that retrieving all fields through
`"fields": ["*"]` doesn't fail because of unsupported value fetchers.
@jtibshirani jtibshirani merged commit 284aaa8 into elastic:7.x Oct 12, 2020
@jtibshirani jtibshirani deleted the value-fetchers branch October 12, 2020 23:15
jtibshirani added a commit to jtibshirani/elasticsearch that referenced this pull request Oct 12, 2020
This PR implements value fetching for the following field types:
* `text` phrase and prefix subfields
* `search_as_you_type`, plus its subfields
* `token_count`, which is implemented by fetching doc values

Supporting these types helps ensure that retrieving all fields through
`"fields": ["*"]` doesn't fail because of unsupported value fetchers.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport >enhancement :Search/Search Search-related issues that do not fall into other categories Team:Search Meta label for search team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants