opensearch-project / asynchronous-search

:arrow_forward: Run queries in the background and retrieve partial results along the way
https://opensearch.org/docs/latest/search-plugins/async/index/
Apache License 2.0
29 stars 49 forks source link

Fix restart bwc #624

Closed finnegancarroll closed 1 month ago

finnegancarroll commented 2 months ago

Description

Full restart bwc only restarts single node.

Related Issues

N/A

Check List

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

mch2 commented 1 month ago

@finnegancarroll is this still required since the version bump?

finnegancarroll commented 1 month ago

Yea this is a fix for the full cluster restart bwc. Behavior should be all nodes restart but currently only one is upgraded. Introduced back here: https://github.com/opensearch-project/asynchronous-search/pull/575/files#diff-49a96e7eea8a94af862798a45174e6ac43eb4f8b4bd40759b5da63ba31ec3ef7L396