Skip to content

[7.x] [Search] Add a new advanced setting searchTimeout (#75728)#77011

Merged
lizozom merged 1 commit intoelastic:7.xfrom
lizozom:backport/7.x/pr-75728
Sep 9, 2020
Merged

[7.x] [Search] Add a new advanced setting searchTimeout (#75728)#77011
lizozom merged 1 commit intoelastic:7.xfrom
lizozom:backport/7.x/pr-75728

Conversation

@lizozom
Copy link
Copy Markdown
Contributor

@lizozom lizozom commented Sep 9, 2020

Backports the following commits to 7.x:

* Add new x-pack advanced setting searchTimeout and use it in the EnhancedSearchInterceptor

* docs

* Rely on server timeout in OSS (?)
Use UI setting in xpack.

* Rename function

* doc

* Remove esShard from client

* cleanup request parameters from FE

* doc

* doc

* Align request parameters on server,
Remove leftover parameters from client
Shim responses for search and msearch routes

* docs
Stop using toSnakeCase
Updates jest tests

* add management docs

* docs

* Remove import

* Break circular dep + fix msearch test

* Remove deleted type

* Fix jest

* Bring toSnakeCase back

* docs

* fix jest

* Fix merge

* Fix types

* Allow timeout to be undefined

* Fix jest test

* Upldate docs

* Fix msearch jest

* docs

* Fix rollup search merge

* docs

Co-authored-by: Lukas Olson <olson.lukas@gmail.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
# Conflicts:
#	src/legacy/core_plugins/elasticsearch/index.js
@lizozom lizozom added the backport This PR is a backport of another PR label Sep 9, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Build metrics

page load bundle size

id value diff baseline
data 1.4MB -1.6KB 1.4MB
dataEnhanced 178.8KB +645.0B 178.1KB
visTypeVega 665.4KB -154.0B 665.5KB
total -1.1KB

distributable file count

id value diff baseline
default 47171 +2 47169
oss 28908 +2 28906
total +4

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

@lizozom lizozom merged commit ba150f7 into elastic:7.x Sep 9, 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.

2 participants