Skip to content

Skip over metadata fields in the field retrieval API. #58710

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

Merged
merged 1 commit into from
Jun 30, 2020

Conversation

jtibshirani
Copy link
Contributor

This avoids unnecessary lookups, since metadata fields don't have _source
values. Plus, many of them already come back as top-level fields in the search
response, such as _id, _index, _routing, etc.

In the future we can consider support for metadata fields like _size, that
must be explicitly retrieved but whose values aren't stored in the document
source.

@jtibshirani jtibshirani added :Search/Search Search-related issues that do not fall into other categories >refactoring labels Jun 30, 2020
@elasticmachine
Copy link
Collaborator

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

@elasticmachine elasticmachine added the Team:Search Meta label for search team label Jun 30, 2020
@jtibshirani jtibshirani requested a review from nik9000 June 30, 2020 01:24
This avoids unnecessary lookups, since metadata fields don't have _source
values.
@jtibshirani jtibshirani merged this pull request into elastic:field-retrieval Jun 30, 2020
@jtibshirani jtibshirani deleted the fetch-meta-fields branch June 30, 2020 19:44
jtibshirani added a commit that referenced this pull request Jun 30, 2020
This avoids unnecessary lookups, since metadata fields don't have _source
values.
jtibshirani added a commit that referenced this pull request Jul 8, 2020
This avoids unnecessary lookups, since metadata fields don't have _source
values.
jtibshirani added a commit that referenced this pull request Jul 14, 2020
This avoids unnecessary lookups, since metadata fields don't have _source
values.
jtibshirani added a commit that referenced this pull request Jul 15, 2020
This avoids unnecessary lookups, since metadata fields don't have _source
values.
jtibshirani added a commit that referenced this pull request Jul 18, 2020
This avoids unnecessary lookups, since metadata fields don't have _source
values.
jtibshirani added a commit that referenced this pull request Jul 21, 2020
This avoids unnecessary lookups, since metadata fields don't have _source
values.
jtibshirani added a commit that referenced this pull request Jul 23, 2020
This avoids unnecessary lookups, since metadata fields don't have _source
values.
jtibshirani added a commit to jtibshirani/elasticsearch that referenced this pull request Jul 27, 2020
This avoids unnecessary lookups, since metadata fields don't have _source
values.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>refactoring :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.

3 participants