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

[BUG] Higher memory consumption when running with JDK 21 #12694

Open
kroepke opened this issue Mar 15, 2024 · 4 comments
Open

[BUG] Higher memory consumption when running with JDK 21 #12694

kroepke opened this issue Mar 15, 2024 · 4 comments
Labels
bug Something isn't working Indexing:Performance Indexing Indexing, Bulk Indexing and anything related to indexing Other Performance This is for any performance related enhancements or bugs

Comments

@kroepke
Copy link

kroepke commented Mar 15, 2024

Describe the bug

We've noticed that some of our integration tests started failing with circuit breaker issues after updating the test container to 2.12.0.
The test in question tries to optimize the size of large bulk indexing requests, but the failure isn't related to the functionality but to overall heap behavior.

Essentially, we are seeing the same issue as described in #12454

Additional testing to isolate the issue leads us to believe that the behavior change is due to the switch to JDK 21.
Please see https://github.com/kroepke/opensearch-jdk21-memory for a smaller test case. It uses bulk indexing mainly to achieve memory pressure.

Flight recorder data yielded nothing obvious except a higher number of class version errors related to Painless components. It's unclear whether this is related or a red herring.

Related component

Other

To Reproduce

Please see https://github.com/kroepke/opensearch-jdk21-memory for a reproduction scenario.

Expected behavior

Switching the default JDK should not massively impact performance.

Additional Details

No response

@kroepke kroepke added bug Something isn't working untriaged labels Mar 15, 2024
@github-actions github-actions bot added the Other label Mar 15, 2024
@bernd
Copy link

bernd commented Mar 15, 2024

We added a JUnit test in addition to the Docker Compose script test that emulates our bulk indexing handling.

kroepke/opensearch-jdk21-memory@7fb6935

@bernd
Copy link

bernd commented Mar 15, 2024

This issue in Elasticsearch seems to be related: elastic/elasticsearch#99592

@andrross andrross added Indexing Indexing, Bulk Indexing and anything related to indexing and removed untriaged labels Mar 20, 2024
@andrross
Copy link
Member

[Triage - attendees 1 2 3 4 5 6]
@bernd Thanks for filing and the detailed reproduction.

@zakisaad
Copy link

This is a real problem for sure on AWS managed OpenSearch (when running 2.13), more context can be seen here: #12454 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Indexing:Performance Indexing Indexing, Bulk Indexing and anything related to indexing Other Performance This is for any performance related enhancements or bugs
Development

No branches or pull requests

5 participants