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(); ) {