Skip to content

Clarify G1GC support on docs #43885

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

Closed
dliappis opened this issue Jul 2, 2019 · 6 comments
Closed

Clarify G1GC support on docs #43885

dliappis opened this issue Jul 2, 2019 · 6 comments
Assignees
Labels
:Core/Infra/Core Core issues without another label >docs General docs changes Team:Core/Infra Meta label for core/infra team Team:Docs Meta label for docs team

Comments

@dliappis
Copy link
Contributor

dliappis commented Jul 2, 2019

As of 6.5.0, G1GC is now supported by Elasticsearch.

We should document this, perhaps in the setup page.

@dliappis dliappis added >docs General docs changes :Core/Infra/Core Core issues without another label labels Jul 2, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@bczifra
Copy link
Member

bczifra commented Jul 8, 2019

Related: #41358

@jrodewig
Copy link
Contributor

jrodewig commented Oct 4, 2019

[docs issue triage]

Leaving this open as it's still relevant.

@rjernst rjernst added Team:Core/Infra Meta label for core/infra team Team:Docs Meta label for docs team labels May 4, 2020
@rjernst rjernst added the needs:triage Requires assignment of a team area label label Dec 3, 2020
@williamrandolph
Copy link
Contributor

@jrodewig @debadair Is this issue still relevant? It looks like we have some discussion of garbage collectors on the page about the serial collector check: https://www.elastic.co/guide/en/elasticsearch/reference/7.10/_use_serial_collector_check.html

That paragraph mentions that we default to using G1GC, and that CMS will only be used if you don't use the JVM shipped with Elasticsearch.

@williamrandolph williamrandolph removed the needs:triage Requires assignment of a team area label label Jan 6, 2021
@jrodewig
Copy link
Contributor

jrodewig commented Jan 6, 2021

Thanks for the ping @williamrandolph.

I'm going to close this issue. With #49123, we now default to G1GC for JDK 14+. With #59667, Elasticsearch 7.11+ ergonomically adjusts G1GC options based on heap size.

While users can manually adjust G1GC options, I don't think we should encourage it or document it in depth. That effort can be covered in #53889.

@jrodewig jrodewig closed this as completed Jan 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Core Core issues without another label >docs General docs changes Team:Core/Infra Meta label for core/infra team Team:Docs Meta label for docs team
Projects
None yet
Development

No branches or pull requests

7 participants