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

[7.x] Reduce memory usage of elasticsearch.index metricset (#16538) #17069

Merged
merged 2 commits into from
Mar 18, 2020

Conversation

ycombinator
Copy link
Contributor

Backports the following commits to 7.x:

@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations (Team:Integrations)

Copy link
Contributor

@blakerouse blakerouse left a comment

Choose a reason for hiding this comment

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

Looks good.

* [Debugging only!] Add charts to see memory usage over time.

THIS COMMIT MUST BE REMOVED AT THE END!!!

* [Debugging only] Disable validation so we can test only index metricset

* Streaming parser

* Removing debugcharts

* Replace maps with structs

* Running go mod tidy

* Pass pointer

* Uncomment code

* Reverting unnecessary changes

* Adding CHANGELOG entry

* Incorporating benchmark test

* Removing unnecessary nil check
@ycombinator ycombinator force-pushed the backport/7.x/pr-16538 branch from c0ce6dc to 9323ec1 Compare March 18, 2020 20:39
@ycombinator ycombinator merged commit 735f601 into elastic:7.x Mar 18, 2020
@ycombinator ycombinator deleted the backport/7.x/pr-16538 branch March 18, 2020 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Team:Integrations Label for the Integrations team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants