Final piece to remove lock Callback

Now that drools-applications no longer uses the lock Callback, it
can be removed from drools-pdp.

Change-Id: Ie82a6d382b54041ab612e73ad8c96176bd01a9f6
Issue-ID: POLICY-872
Signed-off-by: Jim Hahn <jrh3@att.com>

diff --git a/policy-core/src/main/java/org/onap/policy/drools/core/lock/PolicyResourceLockFeatureAPI.java b/policy-core/src/main/java/org/onap/policy/drools/core/lock/PolicyResourceLockFeatureAPI.java
index 1184b01..683dd83 100644 (file)
--- a/policy-core/src/main/java/org/onap/policy/drools/core/lock/PolicyResourceLockFeatureAPI.java
+++ b/policy-core/src/main/java/org/onap/policy/drools/core/lock/PolicyResourceLockFeatureAPI.java
@@ -44,22 +44,6 @@ public interface PolicyResourceLockFeatureAPI extends OrderedService {
     public static OrderedServiceImpl<PolicyResourceLockFeatureAPI> impl =
                     new OrderedServiceImpl<>(PolicyResourceLockFeatureAPI.class);
 
-    /**
-     * Callback that an implementer invokes, asynchronously, when a lock is acquired (or
-     * denied). The implementer invokes the method to indicate that the lock was acquired
-     * (or denied).
-     */
-    @FunctionalInterface
-    public static interface Callback {
-
-        /**
-         * 
-         * @param locked {@code true} if the lock was acquired, {@code false} if the lock
-         *        was denied
-         */
-        public void set(boolean locked);
-    }
-
     /**
          * 
      * Result of a requested operation.

diff --git a/policy-core/src/main/java/org/onap/policy/drools/core/lock/PolicyResourceLockManager.java b/policy-core/src/main/java/org/onap/policy/drools/core/lock/PolicyResourceLockManager.java
index afe1cab..1a94a53 100644 (file)
--- a/policy-core/src/main/java/org/onap/policy/drools/core/lock/PolicyResourceLockManager.java
+++ b/policy-core/src/main/java/org/onap/policy/drools/core/lock/PolicyResourceLockManager.java
@@ -21,10 +21,8 @@
 package org.onap.policy.drools.core.lock;
 
 import java.util.List;
-import java.util.concurrent.Future;
 import java.util.function.Function;
 import java.util.function.Supplier;
-import org.onap.policy.drools.core.lock.PolicyResourceLockFeatureAPI.Callback;
 import org.onap.policy.drools.core.lock.PolicyResourceLockFeatureAPI.OperResult;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -68,19 +66,6 @@ public class PolicyResourceLockManager extends SimpleLockManager {
     protected static void setFactory(Factory factory) {
         PolicyResourceLockManager.factory = factory;
     }
-    
-    /**
-     * This method is here temporarily so-as not to break the drools-applications
-     * build.  It will be removed once drools-applications has been updated.
-     * @param resourceId
-     * @param owner
-     * @param callback
-     * @return nothing; always throws an exception
-     * @throws UnsupportedOperationException
-     */
-    public Future<Boolean> lock(String resourceId, String owner, Callback callback) {
-        throw new UnsupportedOperationException("lock with callback");
-    }
 
     @Override
     public boolean lock(String resourceId, String owner, int holdSec) {