Skip to content

[CI] [9.1] Fix ThreadPoolMergeSchedulerTests testSchedulerCloseWaitsForRunningMerge #130132

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

Open
wants to merge 1 commit into
base: 9.1
Choose a base branch
from

Conversation

albertzaharovits
Copy link
Contributor

This fixes a race condition in the test scenario, between the merge scheduler closing and the merge task being scheduled to run. The test scenario expects that the merge task runs when the scheduler is closed. If the merge scheduler is closed before the merge task is scheduled, the merge task will instead be scheduled as aborted.

Backport of #130078
Fixes: #125236

…rge (elastic#130078)

This fixes a race condition in the test scenario, between the merge
scheduler closing and the merge task being scheduled to run. The test
scenario expects that the merge task runs when the scheduler is closed.
If the merge scheduler is closed before the merge task is scheduled, the
merge task will instead be scheduled as aborted.

Fixes: elastic#125236
@albertzaharovits albertzaharovits self-assigned this Jun 26, 2025
@albertzaharovits albertzaharovits added >test Issues or PRs that are addressing/adding tests :Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. backport auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) v9.1.1 labels Jun 26, 2025
@albertzaharovits
Copy link
Contributor Author

@elasticmachine test this please

1 similar comment
@albertzaharovits
Copy link
Contributor Author

@elasticmachine test this please

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. >test Issues or PRs that are addressing/adding tests v9.1.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant