AbstractResourceManagerFactory Clean Code Hygiene 93/3793/1
authorMarcus G K Williams <marcus.williams@intel.com>
Mon, 1 May 2017 23:28:50 +0000 (16:28 -0700)
committerMarcus G K Williams <marcus.williams@intel.com>
Mon, 1 May 2017 23:37:29 +0000 (16:37 -0700)
Change-Id: I0f10520e86b692ee244a80bbc0047fe220b044f9
Signed-off-by: Marcus G K Williams <marcus.williams@intel.com>
dblib/provider/src/main/java/org/openecomp/sdnc/sli/resource/dblib/factory/AbstractResourceManagerFactory.java

index 29ec061..0f58a3f 100644 (file)
@@ -69,9 +69,9 @@ public abstract class AbstractResourceManagerFactory {
                                ds = CachedDataSourceFactory.createDataSource(config);
                                return ds;
                        } finally {
-                               synchronized(activeTasks){
+                               synchronized(activeTasks) {
                                        activeTasks.remove(this);
-                                       if(activeTasks.isEmpty()){
+                                       if (activeTasks.isEmpty()) {
                                                final Runnable closure = new Runnable() {
 
                                                        public void run() {
@@ -79,20 +79,23 @@ public abstract class AbstractResourceManagerFactory {
                                                                        Thread.sleep(300);
                                                                } catch (Exception e) {
                                                                }
-                                                               synchronized(activeTasks){
+                                                               synchronized(activeTasks) {
                                                                        activeTasks.notifyAll();
                                                                }
                                                        }
                                                };
-                                               if(LOGGER.isDebugEnabled())
-                                                       LOGGER.debug("Completed CachedDataSource.Call and notifyAll from "+ds.getDbConnectionName());
+                                               if (LOGGER.isDebugEnabled()) {
+                                                       LOGGER.debug("Completed CachedDataSource.Call and notifyAll from " + ds.getDbConnectionName());
+                                               }
                                                Thread worker = new Thread(closure);
                                                worker.setDaemon(true);
                                                worker.start();
                                        } else {
-                                               if(LOGGER.isDebugEnabled())
-                                                       if(ds != null)
-                                                       LOGGER.debug("Completed CachedDataSource.Call from "+ds.getDbConnectionName());
+                                               if (LOGGER.isDebugEnabled()) {
+                                                       if (ds != null) {
+                                                               LOGGER.debug("Completed CachedDataSource.Call from " + ds.getDbConnectionName());
+                                                       }
+                                               }
                                        }
                                }
                        }