BZ-1192459: more cleanup on sync; now double checked with cluster setup

kiegroup · Feb 20, 2015 · 2979fc1 · 2979fc1
1 parent 6959d2f
commit 2979fc1
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/.../guvnor-inbox-backend/src/main/java/org/guvnor/inbox/backend/server/InboxBackendImpl.java b/.../guvnor-inbox-backend/src/main/java/org/guvnor/inbox/backend/server/InboxBackendImpl.java
@@ -112,9 +112,9 @@ public void recordUserEditEvent( @Observes final ResourceUpdatedEvent event ) {
      * Helper method to log the opening. Will remove any inbox items that have
      * the same id.
      */
-    private synchronized void recordOpeningEvent( final String itemPath,
-                                                  final String itemName,
-                                                  final String userName ) {
+    private void recordOpeningEvent( final String itemPath,
+                                     final String itemName,
+                                     final String userName ) {
         addToRecentOpened( itemPath, itemName, userName );
         List<InboxEntry> unreadIncoming = removeAnyExisting( itemPath,
                                                              loadIncoming( userName ) );
@@ -125,9 +125,9 @@ private synchronized void recordOpeningEvent( final String itemPath,
      * Helper method to note the event
      */
     //@Override
-    private synchronized void recordUserEditEvent( final String itemPath,
-                                                   final String itemName,
-                                                   final String userName ) {
+    private void recordUserEditEvent( final String itemPath,
+                                      final String itemName,
+                                      final String userName ) {
         addToRecentEdited( itemPath, itemName, userName );
 
         //deliver messages to users inboxes (ie., the edited item is the itme that the current logged in user has edited in the past, or commented on)

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
@@ -164,7 +164,7 @@ public void shutdown() {
     }
 
     @Override
-    public synchronized void startBatch() {
+    public void startBatch() {
         ioService.startBatch( ioService.get( systemRepository.getUri() ).getFileSystem() );
     }
 
@@ -234,7 +234,7 @@ public boolean addConfiguration( final ConfigGroup configGroup ) {
     }
 
     @Override
-    public synchronized boolean updateConfiguration( ConfigGroup configGroup ) {
+    public 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 synchronized boolean updateConfiguration( ConfigGroup configGroup ) {
     }
 
     @Override
-    public synchronized boolean removeConfiguration( final ConfigGroup configGroup ) {
+    public boolean removeConfiguration( final ConfigGroup configGroup ) {
 
         //Invalidate cache if an item has been removed; otherwise cached value is stale
         configuration.remove( configGroup.getType() );