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

Don't throttle recovery indexing operations #9396

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@mikemccand
Contributor

mikemccand commented Jan 23, 2015

Closes #9394

@mikemccand mikemccand added the review label Jan 23, 2015

@bleskes

This comment has been minimized.

Show comment
Hide comment
@bleskes

bleskes Jan 23, 2015

Member

LGTM. I wonder whether it's possible to test this. Also IMHO it should go all the way to 1.4.3.

Member

bleskes commented Jan 23, 2015

LGTM. I wonder whether it's possible to test this. Also IMHO it should go all the way to 1.4.3.

@bleskes

This comment has been minimized.

Show comment
Hide comment
@bleskes

bleskes Jan 23, 2015

Member

apparently the related issue ia already labeled as 1.4.3. So we agree :)

Member

bleskes commented Jan 23, 2015

apparently the related issue ia already labeled as 1.4.3. So we agree :)

@mikemccand

This comment has been minimized.

Show comment
Hide comment
@mikemccand

mikemccand Jan 23, 2015

Contributor

Thanks @bleskes; I think testing this is not easy :(

I'll push all the way back to 1.4.3 (#9394 has the labels).

Contributor

mikemccand commented Jan 23, 2015

Thanks @bleskes; I think testing this is not easy :(

I'll push all the way back to 1.4.3 (#9394 has the labels).

mikemccand added a commit that referenced this pull request Jan 23, 2015

mikemccand added a commit that referenced this pull request Jan 23, 2015

@clintongormley clintongormley removed the review label Mar 19, 2015

@clintongormley clintongormley changed the title from Core: don't throttle recovery indexing operations to Don't throttle recovery indexing operations Jun 6, 2015

mute pushed a commit to mute/elasticsearch that referenced this pull request Jul 29, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment