package org.opendaylight.controller.remote.rpc;
import akka.dispatch.OnComplete;
-import com.google.common.base.Throwables;
+import com.google.common.base.Preconditions;
import com.google.common.util.concurrent.AbstractFuture;
import com.google.common.util.concurrent.CheckedFuture;
import java.util.concurrent.ExecutionException;
import java.util.concurrent.TimeUnit;
import java.util.concurrent.TimeoutException;
-import org.opendaylight.controller.cluster.datastore.node.utils.serialization.NormalizedNodeSerializer;
import org.opendaylight.controller.md.sal.dom.api.DOMRpcException;
import org.opendaylight.controller.md.sal.dom.api.DOMRpcResult;
import org.opendaylight.controller.md.sal.dom.spi.DefaultDOMRpcResult;
import org.opendaylight.controller.remote.rpc.messages.RpcResponse;
+import org.opendaylight.yangtools.yang.common.QName;
import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
private static final Logger LOG = LoggerFactory.getLogger(RemoteDOMRpcFuture.class);
- private RemoteDOMRpcFuture(final Future<Object> future) {
- future.onComplete(new FutureUpdater(), ExecutionContext.Implicits$.MODULE$.global());
+ private final QName rpcName;
+
+ private RemoteDOMRpcFuture(final QName rpcName) {
+ this.rpcName = Preconditions.checkNotNull(rpcName, "rpcName");
+ }
+
+ public static RemoteDOMRpcFuture create(final QName rpcName) {
+ return new RemoteDOMRpcFuture(rpcName);
+ }
+
+ protected void failNow(final Throwable error) {
+ LOG.debug("Failing future {} for rpc {}", this, rpcName, error);
+ setException(error);
}
- public static CheckedFuture<DOMRpcResult, DOMRpcException> from(final Future<Object> future) {
- return new RemoteDOMRpcFuture(future);
+ protected void completeWith(final Future<Object> future) {
+ future.onComplete(new FutureUpdater(), ExecutionContext.Implicits$.MODULE$.global());
}
@Override
} catch (final ExecutionException e) {
throw mapException(e);
} catch (final InterruptedException e) {
- throw Throwables.propagate(e);
+ throw new RemoteDOMRpcException("Interruped while invoking RPC", e);
}
}
} catch (final ExecutionException e) {
throw mapException(e);
} catch (final InterruptedException e) {
- throw Throwables.propagate(e);
+ throw new RemoteDOMRpcException("Interruped while invoking RPC", e);
}
}
- private DOMRpcException mapException(final ExecutionException e) {
- final Throwable cause = e.getCause();
+ private static DOMRpcException mapException(final ExecutionException ex) {
+ final Throwable cause = ex.getCause();
if (cause instanceof DOMRpcException) {
return (DOMRpcException) cause;
}
- return new RemoteDOMRpcException("Exception during invoking RPC", e);
+ return new RemoteDOMRpcException("Exception during invoking RPC", ex);
}
private final class FutureUpdater extends OnComplete<Object> {
@Override
public void onComplete(final Throwable error, final Object reply) throws Throwable {
if (error != null) {
- RemoteDOMRpcFuture.this.setException(error);
+ RemoteDOMRpcFuture.this.failNow(error);
} else if (reply instanceof RpcResponse) {
final RpcResponse rpcReply = (RpcResponse) reply;
- final NormalizedNode<?, ?> result;
- if (rpcReply.getResultNormalizedNode() == null) {
- result = null;
- LOG.debug("Received response for invoke rpc: result is null");
- } else {
- result = NormalizedNodeSerializer.deSerialize(rpcReply.getResultNormalizedNode());
- LOG.debug("Received response for invoke rpc: result is {}", result);
- }
+ final NormalizedNode<?, ?> result = rpcReply.getResultNormalizedNode();
+
+ LOG.debug("Received response for rpc {}: result is {}", rpcName, result);
+
RemoteDOMRpcFuture.this.set(new DefaultDOMRpcResult(result));
+
+ LOG.debug("Future {} for rpc {} successfully completed", RemoteDOMRpcFuture.this, rpcName);
+ } else {
+ RemoteDOMRpcFuture.this.failNow(new IllegalStateException("Incorrect reply type " + reply
+ + "from Akka"));
}
- RemoteDOMRpcFuture.this.setException(new IllegalStateException("Incorrect reply type " + reply
- + "from Akka"));
}
}