Skip to content

Comments

[7.x] [Upgrade Assistant] Open And Close Slight Refactor (#59890)#59992

Merged
jloleysens merged 1 commit intoelastic:7.xfrom
jloleysens:backport/7.x/pr-59890
Mar 12, 2020
Merged

[7.x] [Upgrade Assistant] Open And Close Slight Refactor (#59890)#59992
jloleysens merged 1 commit intoelastic:7.xfrom
jloleysens:backport/7.x/pr-59890

Conversation

@jloleysens
Copy link
Contributor

Backports the following commits to 7.x:

* Refactor: Move checking of closed index to single point

We should rather only check if an index is currently closed the
moment before starting to reindex. We still store a flag to
indicate that we opened an index that was closed, but this
should not be set from the reindex handlers because the reindex
task may only start some time later in which case the closed
index could have been opened and our reindex job will open it
and close it again.

* Added debug log

* Added comment

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@jloleysens jloleysens added the backport This PR is a backport of another PR label Mar 12, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@jloleysens jloleysens merged commit 378e394 into elastic:7.x Mar 12, 2020
@jloleysens jloleysens deleted the backport/7.x/pr-59890 branch March 12, 2020 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants