Remove all_fields:true in favour of fields:* #25551
Labels
>enhancement
help wanted
adoptme
:Search/Search
Search-related issues that do not fall into other categories
Comments
1 task
|
This would be a tremendous help for me in elastic/kibana#12624, currently I can only generate a query against all fields by abusing the |
|
Discussed in FixitFriday. Let's do it! |
jimczi
added a commit
to jimczi/elasticsearch
that referenced
this issue
Jul 21, 2017
This commit changes the way we handle field expansion in `match`, `multi_match` and `query_string` query. The main changes are: - For exact field name, the new behavior is to rewrite to a matchnodocs query when the field name is not found in the mapping. - For partial field names (with `*` suffix), the expansion is done only on `keyword`, `text`, `date` and `number` field types. Other field types are simply ignored. - For all fields (`*`), the expansion is done on accepted field types only (see above) and metadata fields are also filtered. - The `*` notation can also be used to set `default_field` option on`query_string` query. This should replace the needs for the extra option `use_all_fields` which is deprecated in this change. This commit also rewrites simple `*` query to matchalldocs query when all fields are requested (Fixes elastic#25556). The same change should be done on `simple_query_string` for completeness. Relates elastic#25551
clintongormley
added
:Search/Search
|
@jimczi can this now be safely done after your refactor? |
talevy
added
:Search/Search
|
Pinging @elastic/es-search-aggs |
jimczi
added a commit
to jimczi/elasticsearch
that referenced
this issue
Mar 22, 2018
This commit removes some parameters deprecated in 6.x (or 5.x): `use_dismax`, `split_on_whitespace`, `all_fields` and `lowercase_expanded_terms`. Closes elastic#25551
jimczi
added a commit
that referenced
this issue
Mar 22, 2018
This commit removes some parameters deprecated in 6.x (or 5.x): `use_dismax`, `split_on_whitespace`, `all_fields` and `lowercase_expanded_terms`. Closes #25551
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
all_fieldsonly exists because fields:* queries some fields that are not suitable for querying, and the query ultimately fails with a weird exception. We should fixfields:*to do the right thing and remove theall_fieldsoption.The text was updated successfully, but these errors were encountered: