Skip to content
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

Metrics _msearch Timeouts #26449

Closed
kobelb opened this issue Nov 29, 2018 · 1 comment · Fixed by #26510
Closed

Metrics _msearch Timeouts #26449

kobelb opened this issue Nov 29, 2018 · 1 comment · Fixed by #26510
Labels
bug Fixes for quality problems that affect the customer experience Feature:TSVB TSVB (Time Series Visual Builder) Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@kobelb
Copy link
Contributor

kobelb commented Nov 29, 2018

ES master is throwing a 400 Bad Request when the timeout is specified in the _msearch header, and this should be moved to the body and units should be specified.

Doing a quick search, I found the following usages:

  • src/core_plugins/metrics/server/lib/vis_data/table/get_request_params.js
  • src/core_plugins/metrics/server/lib/vis_data/series/get_request_params.js
  • src/core_plugins/metrics/server/lib/vis_data/get_annotations.js
@kobelb kobelb added bug Fixes for quality problems that affect the customer experience Feature:MetricVis Metric visualization feature labels Nov 29, 2018
@timroes timroes added Feature:TSVB TSVB (Time Series Visual Builder) Team:Visualizations Visualization editors, elastic-charts and infrastructure and removed Feature:MetricVis Metric visualization feature labels Dec 3, 2018
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Feature:TSVB TSVB (Time Series Visual Builder) Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants