From 6cec2b32fdd2a9976a8c6e9488608fc84664f16f Mon Sep 17 00:00:00 2001 From: Taka Cho Date: Wed, 17 Jun 2020 10:45:59 -0400 Subject: m2 sonar issues - unused import - logger format - Extract the assignment out of this expression - Replace this if-then-else statement by a single return statement Issue-ID: POLICY-2616 Change-Id: I2d04569a7070f79fc8979274bce2b87b4b2f3e24 Signed-off-by: Taka Cho --- .../src/main/java/org/onap/policy/guard/GuardContext.java | 10 +++------- .../java/org/onap/policy/drools/m2/lock/LockAdjunct.java | 13 ++++--------- 2 files changed, 7 insertions(+), 16 deletions(-) (limited to 'controlloop') diff --git a/controlloop/m2/guard/src/main/java/org/onap/policy/guard/GuardContext.java b/controlloop/m2/guard/src/main/java/org/onap/policy/guard/GuardContext.java index a0d159150..1bfb6ae4c 100644 --- a/controlloop/m2/guard/src/main/java/org/onap/policy/guard/GuardContext.java +++ b/controlloop/m2/guard/src/main/java/org/onap/policy/guard/GuardContext.java @@ -27,19 +27,15 @@ import java.time.Instant; import java.util.HashMap; import java.util.Properties; import java.util.UUID; -import java.util.function.Supplier; import javax.persistence.EntityManager; import javax.persistence.EntityManagerFactory; import javax.persistence.Persistence; import org.drools.core.WorkingMemory; -import org.onap.policy.drools.controller.DroolsController; import org.onap.policy.drools.core.PolicyContainer; import org.onap.policy.drools.core.PolicySession; import org.onap.policy.drools.system.PolicyController; import org.onap.policy.drools.system.PolicyControllerConstants; import org.onap.policy.drools.system.PolicyEngineConstants; -import org.onap.policy.guard.OperationsHistory; -import org.onap.policy.guard.Util; import org.onap.policy.util.DroolsSessionCommonSerializable; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -177,7 +173,7 @@ public class GuardContext implements Serializable { sb.setLength(sb.length() - 2); errorMessage = sb.toString(); disabled = true; - logger.error("Initialization failure: " + errorMessage); + logger.error("Initialization failure: {}", errorMessage); } } @@ -301,7 +297,7 @@ public class GuardContext implements Serializable { if (disabled) { if (errorMessage != null) { - logger.error("Database update skipped: " + errorMessage); + logger.error("Database update skipped: {}", errorMessage); } return false; } @@ -366,7 +362,7 @@ public class GuardContext implements Serializable { final String message, final String outcome) { if (disabled) { if (errorMessage != null) { - logger.error("Database update skipped: " + errorMessage); + logger.error("Database update skipped: {}", errorMessage); } return; } diff --git a/controlloop/m2/lock/src/main/java/org/onap/policy/drools/m2/lock/LockAdjunct.java b/controlloop/m2/lock/src/main/java/org/onap/policy/drools/m2/lock/LockAdjunct.java index e81ab6288..98d82876e 100644 --- a/controlloop/m2/lock/src/main/java/org/onap/policy/drools/m2/lock/LockAdjunct.java +++ b/controlloop/m2/lock/src/main/java/org/onap/policy/drools/m2/lock/LockAdjunct.java @@ -21,7 +21,6 @@ package org.onap.policy.drools.m2.lock; import java.io.Serializable; -import java.util.HashMap; import org.onap.policy.drools.core.lock.Lock; import org.onap.policy.drools.core.lock.LockCallback; @@ -103,14 +102,10 @@ public class LockAdjunct implements Transaction.Adjunct, LockCallback, this.requestor = requestor; // try to allocate a new lock - if ((lock = PolicyEngineConstants.getManager().createLock( - key, ownerKey, 600, this, waitForLock)).isActive()) { - // the lock is good - return true; - } - - // we need to wait for the lock -- return false, - return false; + lock = PolicyEngineConstants.getManager().createLock( + key, ownerKey, 600, this, waitForLock); + // return the boolean value of the lock.isactive + return lock.isActive(); } /*=================================*/ -- cgit 1.2.3-korg