Bug 1729: Make BlockingQueue events in EventDispatcherImpl volatile

Just like queue in PipelineOrchestratorImpl, events in EventDispatcherImpl
should be volatile since it is expected to be modified by different threads.

Ref gerrit in which Madhu fixes a similar issue in PipelineOrchestratorImpl:
https://git.opendaylight.org/gerrit/#/c/10843/

Change-Id: I923cb974788b0425e28af7823bf6cd23aeeb0f5f
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

diff --git a/openstack/net-virt/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/impl/EventDispatcherImpl.java b/openstack/net-virt/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/impl/EventDispatcherImpl.java
index b14daeae39b659e0e7f421e6192c22ec4f240e7b..6f45bdead6f8bb7b238cf2c007cefaa9adb92c3e 100644 (file)
--- a/openstack/net-virt/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/impl/EventDispatcherImpl.java
+++ b/openstack/net-virt/src/main/java/org/opendaylight/ovsdb/openstack/netvirt/impl/EventDispatcherImpl.java
@@ -29,7 +29,7 @@ public class EventDispatcherImpl implements EventDispatcher {
 
     static final Logger logger = LoggerFactory.getLogger(EventDispatcher.class);
     private ExecutorService eventHandler;
-    private BlockingQueue<AbstractEvent> events;
+    private volatile BlockingQueue<AbstractEvent> events;
 
     private AbstractHandler[] handlers;