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 bf3af4a..5044b3e 100644 (file)
--- 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 dee03a1..c7de4c4 100644 (file)
--- 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) {