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

Add Zing JVM to the list of supported garbage collectors. #10051

Closed
wants to merge 1 commit into from

Conversation

dmitriisilin
Copy link
Contributor

Tested benchmark-cli.jar: all the metrics get reproted after applying of the change.
This is not a cosmetic issue. When x-pack metrics are enabled in logstash (xpack.monitoring.elasticsearch.url option) logstash refuses even to start properly since monitoring pipeline keeps crashing (as there are no valid GC data)
Failed to create monitoring event {:message=>"For path: http_address. Map keys: [:stats, :jvm, :os]", :error=>"LogStash::Instrument::MetricStore::MetricNotFound"}

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@dmitriisilin
Copy link
Contributor Author

Hi, can somebody please clarify is there smth I can do for that revision? CLA is sgined and all the checks are green now.

@jsvd
Copy link
Member

jsvd commented Dec 10, 2018

jenkins test this please

Copy link
Member

@jsvd jsvd left a comment

Choose a reason for hiding this comment

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

LGTM

@elasticsearch-bot
Copy link

João Duarte merged this into the following branches!

Branch Commits
master d5b5f9a
6.x 81019d0

@dmitriisilin
Copy link
Contributor Author

Thanks!

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.

None yet

4 participants