commit a6e0f81350133bf694628fc479c32d96439346cc
author Rajan <dhabalia.me@gmail.com> Fri Jun 09 11:49:43 2017 -0700
committer rdhabalia <rdhabalia@yahoo-inc.com> Mon Jun 12 12:20:05 2017 -0700
tree 1375126c46e3e146c2b6120e1fe3eb03b1d0c92d
parent cfc7fa2aef8bb65c841f551c4124c6d7bbba2b40

Fix test: testNeedBrokerDataUpdate loadReport considers absolute percentage change only (#464)

pulsar-broker/src/test/java/com/yahoo/pulsar/broker/loadbalance/ModularLoadManagerImplTest.java

diff --git a/pulsar-broker/src/test/java/com/yahoo/pulsar/broker/loadbalance/ModularLoadManagerImplTest.java b/pulsar-broker/src/test/java/com/yahoo/pulsar/broker/loadbalance/ModularLoadManagerImplTest.java
index b83ec99..c3d60a5 100644
--- a/pulsar-broker/src/test/java/com/yahoo/pulsar/broker/loadbalance/ModularLoadManagerImplTest.java
+++ b/pulsar-broker/src/test/java/com/yahoo/pulsar/broker/loadbalance/ModularLoadManagerImplTest.java
@@ -259,11 +259,18 @@
         lastData.setMsgRateIn(0);
         assert (!needUpdate.get());
 
-        // Minimally test other values to ensure they are included.
+        // Minimally test other absolute values to ensure they are included.
         lastData.getCpu().usage = 100;
         lastData.getCpu().limit = 1000;
         currentData.getCpu().usage = 106;
         currentData.getCpu().limit = 1000;
+        assert (!needUpdate.get());
+        
+        // Minimally test other absolute values to ensure they are included.
+        lastData.getCpu().usage = 100;
+        lastData.getCpu().limit = 1000;
+        currentData.getCpu().usage = 206;
+        currentData.getCpu().limit = 1000;
         assert (needUpdate.get());
 
         lastData.setCpu(new ResourceUsage());