Skip to content

Refactoring: use new Fetch interface that automatically reports and logs errors#11795

Merged
ycombinator merged 11 commits intoelastic:masterfrom
ycombinator:refactoring-mb-es-fetch
Apr 13, 2019
Merged

Refactoring: use new Fetch interface that automatically reports and logs errors#11795
ycombinator merged 11 commits intoelastic:masterfrom
ycombinator:refactoring-mb-es-fetch

Conversation

@ycombinator
Copy link
Copy Markdown
Contributor

@ycombinator ycombinator commented Apr 12, 2019

Refactors code in the elasticsearch Metricbeat module to use the new Fetch interface introduced in #10727.

Note that x-pack code paths in this module were not refactored to use the new interface as we don't want errors from those code paths to be reported into metricbeat-* indices, only logged to Metricbeat logs.

Related: #11763 and #11767.

@ycombinator ycombinator requested a review from a team as a code owner April 12, 2019 22:52
@ycombinator ycombinator changed the title Refactoring: Use new Fetch interface in elasticsearch/shard metricset Refactoring: use new Fetch interface that automatically reports and logs errors Apr 12, 2019
@ycombinator ycombinator added in progress Pull request is currently in progress. Metricbeat Metricbeat module needs_backport PR is waiting to be backported to other branches. refactoring Feature:Stack Monitoring v7.2.0 v8.0.0 labels Apr 12, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/stack-monitoring

@ycombinator ycombinator merged commit 14381b8 into elastic:master Apr 13, 2019
@ycombinator ycombinator deleted the refactoring-mb-es-fetch branch December 25, 2019 11:18
@ycombinator ycombinator removed the needs_backport PR is waiting to be backported to other branches. label Jan 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants