BZ-1192459: quick remediation for deadlock. this should be improved i…

…n order to avoid method level sync. (definitive solution should be careful in order to preserve clustering sync)
kiegroup · Feb 20, 2015 · ecfd8c6 · ecfd8c6
1 parent 0fa1afa
commit ecfd8c6
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/...e-backend/src/main/java/org/guvnor/structure/backend/config/ConfigurationServiceImpl.java b/...e-backend/src/main/java/org/guvnor/structure/backend/config/ConfigurationServiceImpl.java
@@ -234,7 +234,7 @@ public boolean addConfiguration( final ConfigGroup configGroup ) {
     }
 
     @Override
-    public boolean updateConfiguration( ConfigGroup configGroup ) {
+    public synchronized boolean updateConfiguration( ConfigGroup configGroup ) {
         String filename = configGroup.getName().replaceAll( INVALID_FILENAME_CHARS, "_" );
 
         final Path filePath = ioService.get( systemRepository.getUri() ).resolve( filename + configGroup.getType().getExt() );
@@ -258,7 +258,7 @@ public boolean updateConfiguration( ConfigGroup configGroup ) {
     }
 
     @Override
-    public boolean removeConfiguration( final ConfigGroup configGroup ) {
+    public synchronized boolean removeConfiguration( final ConfigGroup configGroup ) {
 
         //Invalidate cache if an item has been removed; otherwise cached value is stale
         configuration.remove( configGroup.getType() );