aboutsummaryrefslogtreecommitdiffstats
path: root/models-interactions
diff options
context:
space:
mode:
authorJim Hahn <jrh3@att.com>2020-06-02 17:17:20 -0400
committerJim Hahn <jrh3@att.com>2020-06-02 17:21:43 -0400
commit4e16bffb503ad08b98abbb70dfe796172cee422e (patch)
treef123671aa5fcc1f34ff757f45c50e9c6cfddcd7a /models-interactions
parentb6bef44f519f1da5d31daf5406a7c57d3d192218 (diff)
Use plain HashMap instead of IdentityHashMap
The javadocs say not to use IdentityHashMap except in special cases. Modified the Actor code to use HashMap instead. Issue-ID: POLICY-2604 Change-Id: Idcae4fe2e1e600fbb01cdcd09b1caddf2820740a Signed-off-by: Jim Hahn <jrh3@att.com>
Diffstat (limited to 'models-interactions')
-rw-r--r--models-interactions/model-actors/actorServiceProvider/src/main/java/org/onap/policy/controlloop/actorserviceprovider/pipeline/FutureManager.java6
-rw-r--r--models-interactions/model-actors/actorServiceProvider/src/main/java/org/onap/policy/controlloop/actorserviceprovider/pipeline/ListenerManager.java5
2 files changed, 6 insertions, 5 deletions
diff --git a/models-interactions/model-actors/actorServiceProvider/src/main/java/org/onap/policy/controlloop/actorserviceprovider/pipeline/FutureManager.java b/models-interactions/model-actors/actorServiceProvider/src/main/java/org/onap/policy/controlloop/actorserviceprovider/pipeline/FutureManager.java
index aac2f77b7..42aaa820d 100644
--- a/models-interactions/model-actors/actorServiceProvider/src/main/java/org/onap/policy/controlloop/actorserviceprovider/pipeline/FutureManager.java
+++ b/models-interactions/model-actors/actorServiceProvider/src/main/java/org/onap/policy/controlloop/actorserviceprovider/pipeline/FutureManager.java
@@ -20,7 +20,8 @@
package org.onap.policy.controlloop.actorserviceprovider.pipeline;
-import java.util.IdentityHashMap;
+import java.util.HashMap;
+import java.util.Map;
import java.util.concurrent.Future;
import lombok.NoArgsConstructor;
@@ -38,8 +39,7 @@ public class FutureManager extends ListenerManager {
* needed if {@link #remove(Future)} is invoked, so that the same {@link Runnable} is
* used each time.
*/
- @SuppressWarnings("rawtypes")
- private final IdentityHashMap<Future, Runnable> future2listener = new IdentityHashMap<>(5);
+ private final Map<Future<?>, Runnable> future2listener = new HashMap<>(5);
/**
* Adds a future that is to be canceled when this controller is stopped. Note: if the
diff --git a/models-interactions/model-actors/actorServiceProvider/src/main/java/org/onap/policy/controlloop/actorserviceprovider/pipeline/ListenerManager.java b/models-interactions/model-actors/actorServiceProvider/src/main/java/org/onap/policy/controlloop/actorserviceprovider/pipeline/ListenerManager.java
index 1d64a8710..b810de393 100644
--- a/models-interactions/model-actors/actorServiceProvider/src/main/java/org/onap/policy/controlloop/actorserviceprovider/pipeline/ListenerManager.java
+++ b/models-interactions/model-actors/actorServiceProvider/src/main/java/org/onap/policy/controlloop/actorserviceprovider/pipeline/ListenerManager.java
@@ -21,7 +21,8 @@
package org.onap.policy.controlloop.actorserviceprovider.pipeline;
import java.util.ArrayList;
-import java.util.IdentityHashMap;
+import java.util.HashMap;
+import java.util.Map;
import lombok.Getter;
import lombok.NoArgsConstructor;
import org.onap.policy.controlloop.actorserviceprovider.Util;
@@ -40,7 +41,7 @@ public class ListenerManager {
/**
* Listeners to be executed when {@link #stop()} is invoked.
*/
- private final IdentityHashMap<Runnable, Void> listeners = new IdentityHashMap<>(5);
+ private final Map<Runnable, Void> listeners = new HashMap<>(5);
/**
* Indicates that operations within the pipeline should stop executing.