Couple of synchronizations for HER-1943 WorkQueue: inconsistent synchronization using fields active, lastCost, peekItem, wakeTime

* WorkQueue.java
    considerActive(), unpeek() - make synchronized
This commit is contained in:
nlevitt
2011-09-12 22:49:32 +00:00
parent 48b0ddfb03
commit 5a4e72d902
@@ -229,7 +229,7 @@ public abstract class WorkQueue implements Frontier.FrontierGroup,
* URI for crawling, and continues until it is deactivated (for example,
* for session-budget reasons).
*/
public void considerActive() {
public synchronized void considerActive() {
if(active) {
return;
}
@@ -336,7 +336,7 @@ public abstract class WorkQueue implements Frontier.FrontierGroup,
* return a different item.
*
*/
public void unpeek(CrawlURI expected) {
public synchronized void unpeek(CrawlURI expected) {
assert expected == peekItem : "unexpected peekItem";
peekItem = null;
}