You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by cu...@apache.org on 2006/02/15 23:06:32 UTC
svn commit: r378104 -
/lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobTracker.java
Author: cutting
Date: Wed Feb 15 14:06:31 2006
New Revision: 378104
URL: http://svn.apache.org/viewcvs?rev=378104&view=rev
Log:
Add missing synchronization.
Modified:
lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobTracker.java
Modified: lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobTracker.java
URL: http://svn.apache.org/viewcvs/lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobTracker.java?rev=378104&r1=378103&r2=378104&view=diff
==============================================================================
--- lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobTracker.java (original)
+++ lucene/hadoop/trunk/src/java/org/apache/hadoop/mapred/JobTracker.java Wed Feb 15 14:06:31 2006
@@ -592,7 +592,7 @@
* A tracker wants to know if any of its Tasks have been
* closed (because the job completed, whether successfully or not)
*/
- public String pollForTaskWithClosedJob(String taskTracker) {
+ public synchronized String pollForTaskWithClosedJob(String taskTracker) {
TreeSet taskIds = (TreeSet) trackerToTaskMap.get(taskTracker);
if (taskIds != null) {
for (Iterator it = taskIds.iterator(); it.hasNext(); ) {