Skip to content

[7.x] [Maps] get max_result_window and max_inner_result_window from index settings (#53500)#53902

Merged
nreese merged 2 commits intoelastic:7.xfrom
nreese:backport/7.x/pr-53500
Jan 2, 2020
Merged

[7.x] [Maps] get max_result_window and max_inner_result_window from index settings (#53500)#53902
nreese merged 2 commits intoelastic:7.xfrom
nreese:backport/7.x/pr-53500

Conversation

@nreese
Copy link
Copy Markdown
Contributor

@nreese nreese commented Jan 2, 2020

Backports the following commits to 7.x:

…ettings (elastic#53500)

* [Maps] pull ES_SIZE_LIMIT and top hits limit from index settings

* get fetch working

* get min values from indicies response

* use indexSettings.maxResultWindow in documents request size

* use max_inner_result_window to define top hits max

* update jest test

* update docs

* more docs changes for top hits

* fix line spacing

* Update docs/maps/maps-aggregations.asciidoc

Co-Authored-By: gchaps <33642766+gchaps@users.noreply.github.com>

* Update docs/maps/vector-layer.asciidoc

Co-Authored-By: gchaps <33642766+gchaps@users.noreply.github.com>

* add api integration test for indexSettings route

* eslint fixes

* review feedback

* display toast on first index settings fetch failure

* clean up

Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@nreese nreese added the backport This PR is a backport of another PR label Jan 2, 2020
@nreese
Copy link
Copy Markdown
Contributor Author

nreese commented Jan 2, 2020

retest

@nreese
Copy link
Copy Markdown
Contributor Author

nreese commented Jan 2, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@nreese nreese merged commit 8312e3d into elastic:7.x Jan 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants