Browse Source

HDFS-9761. Rebalancer sleeps too long between iterations. Contributed by Mingliang Liu.

(cherry picked from commit c6497949e866594050153b953a85c0a1db59d2f8)
(cherry picked from commit a7232f5e1d20faf210f3718c551cbae4edb5933c)
cnauroth 9 years ago
parent
commit
bad8006da1

+ 3 - 0
hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt

@@ -1721,6 +1721,9 @@ Release 2.8.0 - UNRELEASED
     HDFS-9601. NNThroughputBenchmark.BlockReportStats should handle
     HDFS-9601. NNThroughputBenchmark.BlockReportStats should handle
     NotReplicatedYetException on adding block (iwasakims)
     NotReplicatedYetException on adding block (iwasakims)
 
 
+    HDFS-9761. Rebalancer sleeps too long between iterations
+    (Mingliang Liu via cnauroth)
+
 Release 2.7.3 - UNRELEASED
 Release 2.7.3 - UNRELEASED
 
 
   INCOMPATIBLE CHANGES
   INCOMPATIBLE CHANGES

+ 3 - 4
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/balancer/Balancer.java

@@ -673,14 +673,13 @@ public class Balancer {
               // must be an error statue, return.
               // must be an error statue, return.
               return r.exitStatus.getExitCode();
               return r.exitStatus.getExitCode();
             }
             }
-
-            if (!done) {
-              Thread.sleep(sleeptime);
-            }
           } else {
           } else {
             LOG.info("Skipping blockpool " + nnc.getBlockpoolID());
             LOG.info("Skipping blockpool " + nnc.getBlockpoolID());
           }
           }
         }
         }
+        if (!done) {
+          Thread.sleep(sleeptime);
+        }
       }
       }
     } finally {
     } finally {
       for(NameNodeConnector nnc : connectors) {
       for(NameNodeConnector nnc : connectors) {