Skip to content
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

Keep context during reindex's retries #21941

Merged
merged 2 commits into from Dec 2, 2016

Conversation

Projects
None yet
3 participants
@nik9000
Copy link
Contributor

nik9000 commented Dec 2, 2016

This fixes reindex and friend's retries to keep the context.

nik9000 added some commits Nov 9, 2016

Keep context during reindex's retries
This fixes reindex and friend's retries to keep the context.
@nik9000

This comment has been minimized.

Copy link
Contributor Author

nik9000 commented Dec 2, 2016

@javanna or @jaymode, can you have a look at this?

@jaymode

jaymode approved these changes Dec 2, 2016

Copy link
Member

jaymode left a comment

LGTM. Since this is a bugfix, what do you think about backporting to other branches?

@nik9000 nik9000 merged commit 0c724b1 into elastic:master Dec 2, 2016

2 checks passed

CLA Commit author has signed the CLA
Details
elasticsearch-ci Build finished.
Details
@nik9000

This comment has been minimized.

Copy link
Contributor Author

nik9000 commented Dec 2, 2016

Thanks for reviewing @jaymode!

master: 0c724b1
5.x: 9cee9b6
5.1: 248021e

nik9000 added a commit that referenced this pull request Dec 2, 2016

Keep context during reindex's retries (#21941)
* Keep context during reindex's retries

This fixes reindex and friend's retries to keep the context.

* Docs

@nik9000 nik9000 added the v5.1.2 label Dec 2, 2016

nik9000 added a commit that referenced this pull request Dec 2, 2016

Keep context during reindex's retries (#21941)
* Keep context during reindex's retries

This fixes reindex and friend's retries to keep the context.

* Docs

@nik9000 nik9000 added v5.1.1 and removed v5.1.2 labels Dec 5, 2016

@nik9000 nik9000 added the v5.1.1 label Feb 14, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.