fix concurrent access to config file 98/76798/2
authorMatej Perina <matej.perina@pantheon.tech>
Tue, 9 Oct 2018 10:28:27 +0000 (12:28 +0200)
committerMatej Perina <matej.perina@pantheon.tech>
Thu, 11 Oct 2018 07:24:39 +0000 (07:24 +0000)
- calling handleConfigFile from two differrent threads was
  causing ConncurentModificationException in ConfigLoaderImpl
  test

JIRA: BGPCEP-829

Change-Id: I9ada792c98aafc06daa8971b75df22b85b3ff4c1
Signed-off-by: Matej Perina <matej.perina@pantheon.tech>
config-loader/config-loader-impl/src/main/java/org/opendaylight/bgpcep/config/loader/impl/ConfigLoaderImpl.java

index 89882658b4da55db30996711d79f1291ac5ca737..b3a53eec6d38a5634bec28e12984eb57a2852fc4 100644 (file)
@@ -81,7 +81,7 @@ public final class ConfigLoaderImpl implements ConfigLoader, AutoCloseable {
         LOG.info("Config Loader service initiated");
     }
 
-    private void handleConfigFile(final ConfigFileProcessor config, final String filename) {
+    private synchronized void handleConfigFile(final ConfigFileProcessor config, final String filename) {
         final NormalizedNode<?, ?> dto;
         try {
             dto = parseDefaultConfigFile(config, filename);