From fdf0dda3051342c37c04930e15f13e9f5a93bd67 Mon Sep 17 00:00:00 2001 From: Surekha Bejgam Date: Tue, 4 Feb 2014 09:20:32 -0800 Subject: [PATCH] BUG 789: Stats Mgr to skip equality check on flow stats - Statistics Manager runs an equality check on the list of flow statistics received from the protocol plugin with its cached ones to decide whether to update or not, in order to avoid a cluster cache update. In presence of flows on the switch, the flow updated statistics are always different becasue of the milli and nano secs duration fields they contain Change-Id: I8290457c1d0a43294e2449cb16c14d6c1e36481e Signed-off-by: Alessandro Boch Signed-off-by: Surekha Bejgam --- .../statisticsmanager/internal/StatisticsManager.java | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/opendaylight/statisticsmanager/implementation/src/main/java/org/opendaylight/controller/statisticsmanager/internal/StatisticsManager.java b/opendaylight/statisticsmanager/implementation/src/main/java/org/opendaylight/controller/statisticsmanager/internal/StatisticsManager.java index aa6e4ac383..5cd47f2f20 100644 --- a/opendaylight/statisticsmanager/implementation/src/main/java/org/opendaylight/controller/statisticsmanager/internal/StatisticsManager.java +++ b/opendaylight/statisticsmanager/implementation/src/main/java/org/opendaylight/controller/statisticsmanager/internal/StatisticsManager.java @@ -467,11 +467,8 @@ public class StatisticsManager implements IStatisticsManager, IReadServiceListen @Override public void nodeFlowStatisticsUpdated(Node node, List flowStatsList) { - List currentStat = this.flowStatistics.get(node); - // Update cache only if changed to avoid unnecessary cache sync operations - if (! flowStatsList.equals(currentStat)){ - this.flowStatistics.put(node, flowStatsList); - } + // No equality check because duration fields change constantly + this.flowStatistics.put(node, flowStatsList); } @Override -- 2.36.6