Skip to content

[8.x] Remove LongGCDisruption scheme (#115046) #115443

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.

Sign up for GitHub

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

Jump to bottom
Merged
elasticsearchmachine merged 1 commit into from
Oct 23, 2024
Merged

[8.x] Remove LongGCDisruption scheme (#115046) #115443

elasticsearchmachine merged 1 commit into from
Oct 23, 2024

Conversation

@rjernst
Copy link
Member

@rjernst rjernst commented Oct 23, 2024

Backports the following commits to 8.x:

@rjernst
Remove LongGCDisruption scheme (elastic#115046)
96671d6 
Long GC disruption relies on Thread.resume, which is removed in JDK 23.
Tests that use it predate more modern disruption tests. This commit
removes gc disruption and the master disruption tests. Note that tests
relying on this scheme have already not been running since JDK 20 first
deprecated Thread.resume.
@rjernst rjernst added :Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. labels Oct 23, 2024
@elasticsearchmachine elasticsearchmachine mentioned this pull request Oct 23, 2024
Merged
@elasticsearchmachine elasticsearchmachine merged commit f39aba4 into elastic:8.x Oct 23, 2024
14 of 15 checks passed
@rjernst rjernst deleted the backport/8.x/pr-115046 branch October 23, 2024 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. >test Issues or PRs that are addressing/adding tests v8.17.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rjernst @elasticsearchmachine