From 0d0c9edc5ac587a31c274a5acc8e98945442a117 Mon Sep 17 00:00:00 2001 From: Rajan Date: Fri, 9 Jun 2017 11:49:43 -0700 Subject: [PATCH] Fix test: testNeedBrokerDataUpdate loadReport considers absolute percentage change only (#464) --- .../broker/loadbalance/ModularLoadManagerImplTest.java | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) 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 d9df84ba1f1..23ac595ae30 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 @@ -363,11 +363,18 @@ public class ModularLoadManagerImplTest { 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()); -- GitLab