Skip to content

Conversation

@jrodewig
Copy link
Contributor

@jrodewig jrodewig commented Oct 16, 2019

Notes that you should increase the indices.query.bool.max_clause_countsetting if you increase the index.mapping.nested_objects.limit setting.

Closes #46928.

Notes that you should increase the `indices.query.bool.max_clause_count`
setting if you increase the `index.mapping.nested_objects.limit`
setting.

Elasticsearch converts queries on object fields to boolean query
clauses. Increasing both settings ensures you can perform searches on
object fields with a large number of nested properties.

Closes #46928.
@jrodewig jrodewig added >docs General docs changes :Search Foundations/Mapping Index mappings, including merging and defining field types v8.0.0 v7.5.0 v7.6.0 v7.4.2 labels Oct 16, 2019
@jrodewig jrodewig requested a review from jimczi October 16, 2019 17:18
@elasticmachine
Copy link
Collaborator

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

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (>docs)

@thePanz
Copy link

thePanz commented Oct 18, 2019

@jrodewig : the same comment should be added to mapping.total_fields.limit

Copy link
Contributor

@jimczi jimczi left a comment

Choose a reason for hiding this comment

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

Thanks @jrodewig , I left one comment

If you increase this setting, we recommend you also increase the
<<search-settings,`indices.query.bool.max_clause_count`>> setting, which
limits the maximum number of <<query-dsl-bool-query,boolean clauses>> in a query.
====
Copy link
Contributor

Choose a reason for hiding this comment

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

This limit is for the number of nested docs that are allowed on a single nested field. So by default we limit the number of nested documents in the array to 10,000 but that is not related to the boolean max clause.
I'd put the same sentence but in the description of index.mapping.total_fields.limit with a warning that too many fields in mapping and queries should be avoided.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks @jimczi. I've relocated the note to index.mapping.total_fields.limit with a2eff08.

@jrodewig jrodewig changed the title [DOCS] Note clause limit in index.mapping.nested_objects.limit docs [DOCS] Note clause limit in index.mapping.total_fields.limit docs Oct 18, 2019
Copy link
Contributor

@jimczi jimczi left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @jrodewig

@jrodewig
Copy link
Contributor Author

@elasticmachine run elasticsearch-ci/oss-distro-docs

@jrodewig jrodewig merged commit 6accc46 into elastic:master Oct 18, 2019
@jrodewig jrodewig deleted the doc-mapping-boolean-limit branch October 18, 2019 14:20
jrodewig added a commit that referenced this pull request Oct 18, 2019
…48153)

Notes that you should increase the `indices.query.bool.max_clause_count`
setting if you increase the `index.mapping.total_fields.limit`
setting.

Closes #46928.
jrodewig added a commit that referenced this pull request Oct 18, 2019
…48153)

Notes that you should increase the `indices.query.bool.max_clause_count`
setting if you increase the `index.mapping.total_fields.limit`
setting.

Closes #46928.
jrodewig added a commit that referenced this pull request Oct 18, 2019
…48153)

Notes that you should increase the `indices.query.bool.max_clause_count`
setting if you increase the `index.mapping.total_fields.limit`
setting.

Closes #46928.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>docs General docs changes :Search Foundations/Mapping Index mappings, including merging and defining field types v7.4.2 v7.5.0 v7.6.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DOCS] _exists_ returns too_many_clauses exception

5 participants