Skip to content

Remove the support for fielddata_fields #19027

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

Closed
jimczi opened this issue Jun 22, 2016 · 7 comments
Closed

Remove the support for fielddata_fields #19027

jimczi opened this issue Jun 22, 2016 · 7 comments
Labels
>breaking help wanted adoptme :Search/Search Search-related issues that do not fall into other categories

Comments

@jimczi
Copy link
Contributor

jimczi commented Jun 22, 2016

The parameter fielddata_fields is deprecated (#18943) but we continue to fallback to the fielddata cache when the docvalues are not enabled on the field.
We should remove the ability to retrieve the fields through the fielddata cache because the output is not equivalent to the docvalue and may greatly differ from the stored field.

@jimczi jimczi added >breaking discuss :Search/Search Search-related issues that do not fall into other categories labels Jun 22, 2016
@rashidkpc
Copy link

Note that Kibana makes use of fielddata_fields for retrieving the millisecond value of date fields in documents. Is there an alternative for that purpose?

@rashidkpc
Copy link

I guess docvalue_fields? Is that going to break on old indices that don't have docvalues? Is there a way to have a transparent fallback here?

@clintongormley
Copy link
Contributor

clintongormley commented Jul 1, 2016

I guess docvalue_fields? Is that going to break on old indices that don't have docvalues? Is there a way to have a transparent fallback here?

We need to fallback to fielddata for 5.x (at least for old indices), can be removed in 6.0

@jimczi
Copy link
Contributor Author

jimczi commented Jul 1, 2016

discussed in FixItFriday, the fallback to fielddata for 5.x is only for keyword/text fields. It won't work if the field is numeric since the fielddata for numerics has been removed entirely.

Note that Kibana makes use of fielddata_fields for retrieving the millisecond value of date fields in documents. Is there an alternative for that purpose?

You'll have to ensure that the doc_values are activated on the field otherwise it won't work. This is a side effect of the fielddata removal for numeric fields. Note that this has nothing to do with this issue, this is how the 5.x works and this is not going to change.

@clintongormley
Copy link
Contributor

Fielddata for numeric fields were deprecated in 2.0 and removed in 5.0, so doc values would be enabled for any numeric field in an index created in 2.x, unless the user changed the mappings.

So we're good to remove fielddata_fields, and kibana will need to change their use of fielddata_fields to docvalue_fields

@colings86 colings86 added the help wanted adoptme label Mar 31, 2017
yackushevas added a commit to yackushevas/grafana that referenced this issue May 8, 2017
The parameter fielddata_fields is deprecated and removed in 5.0 elastic/elasticsearch#19027
torkelo pushed a commit to grafana/grafana that referenced this issue May 9, 2017
The parameter fielddata_fields is deprecated and removed in 5.0 elastic/elasticsearch#19027
@colings86
Copy link
Contributor

fielddata_fields were deprecated in 5.0 so can be removed in 6.0

@jimczi
Copy link
Contributor Author

jimczi commented Aug 21, 2017

Closed by #25537

@jimczi jimczi closed this as completed Aug 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>breaking help wanted adoptme :Search/Search Search-related issues that do not fall into other categories
Projects
None yet
Development

No branches or pull requests

4 participants