From: Giovanni Meo Date: Fri, 13 Sep 2013 15:41:51 +0000 (+0200) Subject: Addressing gerrit 1182 Madhu's comment X-Git-Tag: releasepom-0.1.0~93 X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=commitdiff_plain;h=cb6fd7676c31ccae1f13077a48bc2e36e67089b4;ds=inline 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 --- 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(); } /**