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 nested and object fields to field capabilities response #33803

Merged
merged 6 commits into from
Sep 26, 2018

Conversation

jimczi
Copy link
Contributor

@jimczi jimczi commented Sep 18, 2018

This commit adds nested and object fields to the field capabilities response.

Closes #33237

This commit adds nested and object fields to the field capabilities response.

Closes elastic#33237
@jimczi jimczi added >enhancement :Search/Search Search-related issues that do not fall into other categories v7.0.0 v6.5.0 labels Sep 18, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search-aggs

@jimczi
Copy link
Contributor Author

jimczi commented Sep 18, 2018

\cc @costin

@costin
Copy link
Member

costin commented Sep 22, 2018

FTR, I've used this PR in SQL and it behaved as expected.

while (dotIndex > -1) {
String parentField = ft.name().substring(0, dotIndex);
ObjectMapper mapper = mapperService.getObjectMapper(parentField);
if (mapper != null) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was wondering whether this condition may ever be false, maybe mention sub fields in a comment? Should we also break as soon as responseMap.containsKey(parentField) is true?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we also break as soon as responseMap.containsKey(parentField) is true?

If we start from the leaf, yes, I pushed
235df36 to reverse the iteration order.

@jimczi jimczi merged commit a255880 into elastic:master Sep 26, 2018
@jimczi jimczi deleted the field_caps_with_objects branch September 26, 2018 06:59
jimczi added a commit that referenced this pull request Sep 26, 2018
This commit adds nested and object fields to the field capabilities response.

Closes #33237
costin added a commit to costin/elasticsearch that referenced this pull request Oct 22, 2018
Extend querying support on multiple indices from being strictly
identical to being just compatible.
Use FieldCapabilities API (extended through elastic#33803) for mapping merging.

Close elastic#31837 elastic#31611
costin added a commit that referenced this pull request Oct 23, 2018
Extend querying support on multiple indices from being strictly
identical to being just compatible.
Use FieldCapabilities API (extended through #33803) for mapping merging.

Close #31837 #31611
costin added a commit that referenced this pull request Oct 23, 2018
Extend querying support on multiple indices from being strictly
identical to being just compatible.
Use FieldCapabilities API (extended through #33803) for mapping merging.

Close #31837 #31611

(cherry picked from commit ca6808e)
kcm pushed a commit that referenced this pull request Oct 30, 2018
This commit adds nested and object fields to the field capabilities response.

Closes #33237
kcm pushed a commit that referenced this pull request Oct 30, 2018
Extend querying support on multiple indices from being strictly
identical to being just compatible.
Use FieldCapabilities API (extended through #33803) for mapping merging.

Close #31837 #31611
@colings86 colings86 removed the v7.0.0 label Feb 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>enhancement :Search/Search Search-related issues that do not fall into other categories v6.5.0 v7.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants