浏览代码

Revert "MAPREDUCE-7250. FrameworkUploader skip replication check if timeout is 0."

This reverts commit 8d82c441029522f447df1ab2b6d770ca15653207.
prabhujoseph 5 年之前
父节点
当前提交
304e75a48b

+ 14 - 18
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-uploader/src/main/java/org/apache/hadoop/mapred/uploader/FrameworkUploader.java

@@ -298,24 +298,20 @@ public class FrameworkUploader implements Runnable {
       fileSystem.setReplication(targetPath, finalReplication);
       LOG.info("Set replication to " +
           finalReplication + " for path: " + targetPath);
-      if (timeout == 0) {
-        LOG.info("Timeout is set to 0. Skipping replication check.");
-      } else {
-        long startTime = System.currentTimeMillis();
-        long endTime = startTime;
-        long currentReplication = 0;
-        while(endTime - startTime < timeout * 1000 &&
-             currentReplication < acceptableReplication) {
-          Thread.sleep(1000);
-          endTime = System.currentTimeMillis();
-          currentReplication = getSmallestReplicatedBlockCount();
-        }
-        if (endTime - startTime >= timeout * 1000) {
-          LOG.error(String.format(
-              "Timed out after %d seconds while waiting for acceptable" +
-                  " replication of %d (current replication is %d)",
-              timeout, acceptableReplication, currentReplication));
-        }
+      long startTime = System.currentTimeMillis();
+      long endTime = startTime;
+      long currentReplication = 0;
+      while(endTime - startTime < timeout * 1000 &&
+           currentReplication < acceptableReplication) {
+        Thread.sleep(1000);
+        endTime = System.currentTimeMillis();
+        currentReplication = getSmallestReplicatedBlockCount();
+      }
+      if (endTime - startTime >= timeout * 1000) {
+        LOG.error(String.format(
+            "Timed out after %d seconds while waiting for acceptable" +
+                " replication of %d (current replication is %d)",
+            timeout, acceptableReplication, currentReplication));
       }
     } else {
       LOG.info("Cannot set replication to " +