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

HER-2065 synchronize on inactiveQueuesByPrecedence inside of synchronize... #61

Merged
merged 1 commit into from
Apr 25, 2014

Conversation

nlevitt
Copy link
Contributor

@nlevitt nlevitt commented Apr 23, 2014

...d(WorkQueue), not the other way around, to avoid deadlock

@nlevitt
Copy link
Contributor Author

nlevitt commented Apr 23, 2014

…r, and move updateHighestWaiting() up to keep it inside the block; now the call to deactivateQueue() is not in the synchronized block, so the deadlock should be avoided (fix suggested by kenji)
kngenie added a commit that referenced this pull request Apr 25, 2014
HER-2065 synchronize on inactiveQueuesByPrecedence inside of synchronize...
@kngenie kngenie merged commit 50a38e1 into internetarchive:master Apr 25, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants