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.

Sign up for GitHub

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

Cache index lock sometimes not released #1048

Closed
jgosmann opened this issue Apr 29, 2016 · 1 comment
Closed

Cache index lock sometimes not released #1048

jgosmann opened this issue Apr 29, 2016 · 1 comment
Labels
bug
Milestone
2.1.1 release

Comments

@jgosmann
Copy link
Collaborator

Instead of relying on __del__, context managers should be used. This should also resolve #1041.
@jgosmann jgosmann self-assigned this Apr 29, 2016
@jgosmann jgosmann added the bug label Apr 29, 2016
@jgosmann jgosmann added this to the 2.1.1 release milestone Apr 29, 2016
jgosmann added a commit that referenced this issue May 2, 2016
@jgosmann
Use cache as context manager.
44eb9cb 
Do not rely on __del__ which can cause problems.

Addresses #1041, #1048.
@jgosmann jgosmann mentioned this issue May 2, 2016
Merged
@jgosmann jgosmann removed their assignment May 2, 2016
jgosmann added a commit that referenced this issue May 3, 2016
@jgosmann
Use cache as context manager.
326658b 
Do not rely on __del__ which can cause problems.

Addresses #1041, #1048.
hunse pushed a commit that referenced this issue May 25, 2016
@jgosmann @hunse
Use cache as context manager
82afd45 
Do not rely on __del__ which can cause problems.

Addresses #1041, #1048.
@jgosmann
Copy link
Collaborator Author

jgosmann commented Jun 7, 2016

Should be fixed with #1053.

@jgosmann jgosmann closed this as completed Jun 7, 2016
This was referenced Jun 22, 2016
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Milestone
2.1.1 release
Development

No branches or pull requests
1 participant
@jgosmann