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

Jump to bottom

Unregistering snapshot repositories causes thread leaks #6181

Closed
imotov opened this issue May 15, 2014 · 3 comments · Fixed by #6182
Closed

Unregistering snapshot repositories causes thread leaks #6181

imotov opened this issue May 15, 2014 · 3 comments · Fixed by #6182
Assignees
@imotov
Labels
>bug v1.2.0 v2.0.0-beta1

Comments

@imotov
Copy link
Contributor

imotov commented May 15, 2014

No description provided.

@imotov imotov self-assigned this May 15, 2014
@imotov imotov mentioned this issue May 15, 2014
Merged
imotov added a commit to imotov/elasticsearch that referenced this issue May 16, 2014
@imotov
Switch to shared thread pool for all snapshot repositories
c207135 
 Closes elastic#6181
imotov added a commit that referenced this issue May 16, 2014
@imotov
Switch to shared thread pool for all snapshot repositories
a4d9479 
 Closes #6181
@s1monw
Copy link
Contributor

s1monw commented May 18, 2014

should we get this into 1.1.2 as well?

@imotov imotov added v1.1.2 and removed v1.1.2 labels May 19, 2014
@imotov
Copy link
Contributor Author

imotov commented May 19, 2014

It would be difficult because of all deprecated gateways that are still there.

@s1monw
Copy link
Contributor

s1monw commented May 19, 2014

oh yeah I see makes sense - in that case no 1.1.2

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

@imotov imotov

Labels
>bug v1.2.0 v2.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Switch to shared thread pool for all snapshot repositories imotov/elasticsearch
2 participants
@imotov @s1monw