From cb6fd7676c31ccae1f13077a48bc2e36e67089b4 Mon Sep 17 00:00:00 2001 From: Giovanni Meo Date: Fri, 13 Sep 2013 17:41:51 +0200 Subject: [PATCH 1/1] Addressing gerrit 1182 Madhu's comment - Making sure the executor service is available before the loadconfig gets called, just in case of some race condition Change-Id: Ib1d84c33536a68806d281d23855a6c0a5be2dd23 Signed-off-by: Giovanni Meo --- .../internal/ForwardingRulesManager.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/opendaylight/forwardingrulesmanager/implementation/src/main/java/org/opendaylight/controller/forwardingrulesmanager/internal/ForwardingRulesManager.java b/opendaylight/forwardingrulesmanager/implementation/src/main/java/org/opendaylight/controller/forwardingrulesmanager/internal/ForwardingRulesManager.java index 0ac375668b..d08b24d078 100644 --- a/opendaylight/forwardingrulesmanager/implementation/src/main/java/org/opendaylight/controller/forwardingrulesmanager/internal/ForwardingRulesManager.java +++ b/opendaylight/forwardingrulesmanager/implementation/src/main/java/org/opendaylight/controller/forwardingrulesmanager/internal/ForwardingRulesManager.java @@ -2517,6 +2517,9 @@ public class ForwardingRulesManager implements // Initialize graceful stop flag stopping = false; + // Allocate the executor service + this.executor = Executors.newSingleThreadExecutor(); + // Start event handler thread frmEventHandler.start(); @@ -2527,9 +2530,6 @@ public class ForwardingRulesManager implements if (staticFlows.isEmpty()) { loadFlowConfiguration(); } - - // Allocate the executor service - this.executor = Executors.newSingleThreadExecutor(); } /** -- 2.36.6