Skip to content

Wait for task on master in testGetMappingsCancellation (#91709) #91916

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
merged 1 commit into from
Nov 24, 2022
Merged

Wait for task on master in testGetMappingsCancellation (#91709) #91916

merged 1 commit into from
Nov 24, 2022

Conversation

@pxsalehi
Copy link
Member

@pxsalehi pxsalehi commented Nov 24, 2022 •
edited
Loading

Forgotten backport of #91709!

@pxsalehi pxsalehi added >test Issues or PRs that are addressing/adding tests :Distributed Coordination/Task Management Issues for anything around the Tasks API - both persistent and node level. backport auto-backport-and-merge v7.17.8 labels Nov 24, 2022
@pxsalehi pxsalehi requested a review from fcofdez November 24, 2022 16:36
fcofdez
fcofdez approved these changes Nov 24, 2022
View reviewed changes
Copy link
Contributor

@fcofdez fcofdez left a comment

Choose a reason for hiding this comment

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

LGTM

@pxsalehi
Copy link
Member Author

pxsalehi commented Nov 24, 2022

Thanks Francisco!

@pxsalehi pxsalehi merged commit 744ed93 into elastic:8.6 Nov 24, 2022
@pxsalehi pxsalehi mentioned this pull request Nov 24, 2022
Merged
pxsalehi added a commit to pxsalehi/elasticsearch that referenced this pull request Nov 24, 2022
@pxsalehi
Wait for task on master in testGetMappingsCancellation (elastic#91709) (
dde431d 
elastic#91916)
elasticsearchmachine pushed a commit that referenced this pull request Nov 25, 2022
@pxsalehi
[7.17] Wait for task on master in testGetMappingsCancellation (#91709) (
cd96706 
#91916) (#91926)

* Wait for task on master in testGetMappingsCancellation (#91709) (#91916)

* replace List.of usage
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fcofdez fcofdez fcofdez approved these changes

Assignees

No one assigned

Labels

backport :Distributed Coordination/Task Management Issues for anything around the Tasks API - both persistent and node level. >test Issues or PRs that are addressing/adding tests v7.17.8 v8.6.1

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@pxsalehi @fcofdez @elasticsearchmachine