Sonar issue fix in LockCleanUpDaemon

Issue-ID: MUSIC-583
Change-Id: Icc16848de6eaa6ac31112084da012fda9adfdf03
Signed-off-by: ezhil <ezhrajam@in.ibm.com>

diff --git a/music-core/src/main/java/org/onap/music/lockingservice/cassandra/LockCleanUpDaemon.java b/music-core/src/main/java/org/onap/music/lockingservice/cassandra/LockCleanUpDaemon.java
index 492a48f..623a07c 100644 (file)
--- a/music-core/src/main/java/org/onap/music/lockingservice/cassandra/LockCleanUpDaemon.java
+++ b/music-core/src/main/java/org/onap/music/lockingservice/cassandra/LockCleanUpDaemon.java
@@ -23,12 +23,10 @@
 package org.onap.music.lockingservice.cassandra;
 
 import java.util.HashSet;
-import java.util.List;
 import java.util.Set;
 import org.onap.music.datastore.MusicDataStoreHandle;
 import org.onap.music.datastore.PreparedQueryObject;
 import org.onap.music.eelf.logging.EELFLoggerDelegate;
-import org.onap.music.exceptions.MusicQueryException;
 import org.onap.music.exceptions.MusicServiceException;
 import org.onap.music.main.MusicCore;
 import org.onap.music.main.MusicUtil;
@@ -58,7 +56,8 @@ public class LockCleanUpDaemon extends Thread {
             try {
                 Thread.sleep(MusicUtil.getLockDaemonSleepTimeMs());
             } catch (InterruptedException e) {
-                e.printStackTrace();
+                logger.warn(EELFLoggerDelegate.applicationLogger, "Interrupted exception", e);
+
             }
         }
     }
@@ -110,12 +109,14 @@ public class LockCleanUpDaemon extends Thread {
     
     
     private void deleteLockIfStale(String lockTable, Row lock) throws MusicServiceException {
-        if (lock.isNull("createtime") && lock.isNull("acquiretime")) {
+        final String CREATETIME="createtime";
+        final String ACQUIRETIME="acquiretime";
+        if (lock.isNull(CREATETIME) && lock.isNull("acquiretime")) {
             return;
         }
 
-        long createTime = lock.isNull("createtime") ? 0 : Long.parseLong(lock.getString("createtime"));
-        long acquireTime = lock.isNull("acquiretime") ? 0 : Long.parseLong(lock.getString("acquiretime"));
+        long createTime = lock.isNull(CREATETIME) ? 0 : Long.parseLong(lock.getString(CREATETIME));
+        long acquireTime = lock.isNull(ACQUIRETIME) ? 0 : Long.parseLong(lock.getString(ACQUIRETIME));
         long row_access_time = Math.max(createTime, acquireTime);
         if (System.currentTimeMillis() > row_access_time + MusicUtil.getDefaultLockLeasePeriod()) {
             logger.info(EELFLoggerDelegate.applicationLogger, "Stale lock detected and being removed: " + lock);