Skip to content

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

Merged
ycombinator merged 2 commits intoelastic:7.xfrom
ycombinator:backport/7.x/pr-16538
Mar 18, 2020
Merged

[7.x] Reduce memory usage of elasticsearch.index metricset (#16538)#17069
ycombinator merged 2 commits intoelastic:7.xfrom
ycombinator:backport/7.x/pr-16538

Conversation

@ycombinator
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/integrations (Team:Integrations)

Copy link
Copy Markdown
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 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