*/
package org.opendaylight.controller.cluster.datastore;
-import com.google.common.base.Optional;
+import akka.actor.ActorRef;
import java.util.EventListener;
-import java.util.Map.Entry;
import javax.annotation.concurrent.GuardedBy;
+import org.opendaylight.controller.cluster.datastore.messages.ListenerRegistrationMessage;
import org.opendaylight.yangtools.concepts.ListenerRegistration;
-import org.opendaylight.yangtools.yang.data.api.schema.tree.DataTreeCandidate;
-abstract class DelayedListenerRegistration<L extends EventListener, R> implements ListenerRegistration<L> {
- private final R registrationMessage;
- private volatile ListenerRegistration<L> delegate;
+class DelayedListenerRegistration<L extends EventListener, M extends ListenerRegistrationMessage>
+ implements ListenerRegistration<L> {
+ private final M registrationMessage;
+ private final ActorRef registrationActor;
@GuardedBy("this")
private boolean closed;
- protected DelayedListenerRegistration(R registrationMessage) {
+ protected DelayedListenerRegistration(M registrationMessage, ActorRef registrationActor) {
this.registrationMessage = registrationMessage;
+ this.registrationActor = registrationActor;
}
- R getRegistrationMessage() {
+ M getRegistrationMessage() {
return registrationMessage;
}
- ListenerRegistration<L> getDelegate() {
- return delegate;
- }
-
- synchronized <LR extends ListenerRegistration<L>> void createDelegate(
- final LeaderLocalDelegateFactory<R, LR, Optional<DataTreeCandidate>> factory) {
+ synchronized void doRegistration(final AbstractDataListenerSupport<L, M> support) {
if (!closed) {
- final Entry<LR, Optional<DataTreeCandidate>> res = factory.createDelegate(registrationMessage);
- this.delegate = res.getKey();
+ support.doRegistration(registrationMessage, registrationActor);
}
}
@Override
public L getInstance() {
- final ListenerRegistration<L> d = delegate;
- return d == null ? null : (L)d.getInstance();
+ // ObjectRegistration annotates this method as @Nonnull but we could return null if the delegate is not set yet.
+ // In reality, we do not and should not ever call this method on DelayedListenerRegistration instances anyway
+ // but, since we have to provide an implementation to satisfy the interface, we throw
+ // UnsupportedOperationException to honor the API contract of not returning null and to avoid a FindBugs error
+ // for possibly returning null.
+ throw new UnsupportedOperationException(
+ "getInstance should not be called on this instance since it could be null");
}
@Override
public synchronized void close() {
- if (!closed) {
- closed = true;
- if (delegate != null) {
- delegate.close();
- }
- }
+ closed = true;
}
}