From: Taka Cho <takamune.cho@att.com>
Date: Mon, 22 Jun 2020 13:52:30 +0000 (-0400)
Subject: removed unused methods parameter
X-Git-Tag: 1.7.0~20
X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=c951a18cd53cbd1d2c26a78b29b941c64150839f;p=policy%2Fdrools-pdp.git

removed unused methods parameter

Unused parameters are misleading.
Whatever the values passed to such
parameters, the behavior will be the same.

Issue-ID: POLICY-2616
Change-Id: I9a595dca81ce061846fa53adf8086130ffd31a74
Signed-off-by: Taka Cho <takamune.cho@att.com>
---

diff --git a/feature-server-pool/src/main/java/org/onap/policy/drools/serverpool/Bucket.java b/feature-server-pool/src/main/java/org/onap/policy/drools/serverpool/Bucket.java
index bf3af4a8..5044b3e9 100644
--- a/feature-server-pool/src/main/java/org/onap/policy/drools/serverpool/Bucket.java
+++ b/feature-server-pool/src/main/java/org/onap/policy/drools/serverpool/Bucket.java
@@ -874,7 +874,7 @@ public class Bucket {
                                          || thisServer == secondaryBackup));
 
         // invoke 'TargetLock' directly
-        TargetLock.auditBucket(this, isOwner, isBackup);
+        TargetLock.auditBucket(this, isOwner);
         for (ServerPoolApi feature : ServerPoolApi.impl.getList()) {
             feature.auditBucket(this, isOwner, isBackup);
         }
diff --git a/feature-server-pool/src/main/java/org/onap/policy/drools/serverpool/TargetLock.java b/feature-server-pool/src/main/java/org/onap/policy/drools/serverpool/TargetLock.java
index dee03a1b..c7de4c43 100644
--- a/feature-server-pool/src/main/java/org/onap/policy/drools/serverpool/TargetLock.java
+++ b/feature-server-pool/src/main/java/org/onap/policy/drools/serverpool/TargetLock.java
@@ -660,9 +660,8 @@ public class TargetLock implements Lock, Serializable {
      *
      * @param bucket the bucket to audit
      * @param owner 'true' if the current host owns the bucket
-     * @param backup 'true' if the current host is a backup for the bucket
      */
-    static void auditBucket(Bucket bucket, boolean isOwner, boolean isBackup) {
+    static void auditBucket(Bucket bucket, boolean isOwner) {
         if (!isOwner) {
             // we should not have any 'TargetLock' adjuncts
             if (bucket.removeAdjunct(LocalLocks.class) != null) {