2 * Copyright (c) 2016 Cisco Systems, Inc. and others. All rights reserved.
4 * This program and the accompanying materials are made available under the
5 * terms of the Eclipse Public License v1.0 which accompanies this distribution,
6 * and is available at http://www.eclipse.org/legal/epl-v10.html
8 package org.opendaylight.controller.cluster.access.client;
10 import akka.actor.ActorRef;
11 import com.google.common.base.Preconditions;
12 import com.google.common.base.Verify;
13 import com.google.common.collect.Iterables;
14 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
15 import java.util.ArrayDeque;
16 import java.util.Iterator;
17 import java.util.Optional;
18 import java.util.Queue;
19 import javax.annotation.concurrent.NotThreadSafe;
20 import org.opendaylight.controller.cluster.access.concepts.Request;
21 import org.opendaylight.controller.cluster.access.concepts.RequestEnvelope;
22 import org.opendaylight.controller.cluster.access.concepts.RequestException;
23 import org.opendaylight.controller.cluster.access.concepts.Response;
24 import org.opendaylight.controller.cluster.access.concepts.ResponseEnvelope;
25 import org.slf4j.Logger;
26 import org.slf4j.LoggerFactory;
29 * This queue is internally split into two queues for performance reasons, both memory efficiency and copy
33 * Entries are always appended to the end, but then they are transmitted to the remote end and do not necessarily
34 * complete in the order in which they were sent -- hence the head of the queue does not increase linearly,
35 * but can involve spurious removals of non-head entries.
38 * For memory efficiency we want to pre-allocate both queues -- which points to ArrayDeque, but that is very
39 * inefficient when entries are removed from the middle. In the typical case we expect the number of in-flight
40 * entries to be an order of magnitude lower than the number of enqueued entries, hence the split.
43 * Note that in transient case of reconnect, when the backend gives us a lower number of maximum in-flight entries
44 * than the previous incarnation, we may end up still moving the pending queue -- but that is a very exceptional
45 * scenario, hence we consciously ignore it to keep the design relatively simple.
48 * This class is not thread-safe, as it is expected to be guarded by {@link AbstractClientConnection}.
50 * @author Robert Varga
53 abstract class TransmitQueue {
54 static final class Halted extends TransmitQueue {
55 Halted(final int targetDepth) {
60 int canTransmitCount(final int inflightSize) {
65 TransmittedConnectionEntry transmit(final ConnectionEntry entry, final long now) {
66 throw new UnsupportedOperationException("Attempted to transmit on a halted queue");
70 static final class Transmitting extends TransmitQueue {
71 private final BackendInfo backend;
72 private long nextTxSequence;
74 Transmitting(final int targetDepth, final BackendInfo backend) {
76 this.backend = Preconditions.checkNotNull(backend);
80 int canTransmitCount(final int inflightSize) {
81 return backend.getMaxMessages() - inflightSize;
85 TransmittedConnectionEntry transmit(final ConnectionEntry entry, final long now) {
86 final RequestEnvelope env = new RequestEnvelope(entry.getRequest().toVersion(backend.getVersion()),
87 backend.getSessionId(), nextTxSequence++);
89 final TransmittedConnectionEntry ret = new TransmittedConnectionEntry(entry, env.getSessionId(),
90 env.getTxSequence(), now);
91 backend.getActor().tell(env, ActorRef.noSender());
96 private static final Logger LOG = LoggerFactory.getLogger(TransmitQueue.class);
98 private final ArrayDeque<TransmittedConnectionEntry> inflight = new ArrayDeque<>();
99 private final ArrayDeque<ConnectionEntry> pending = new ArrayDeque<>();
100 private final ProgressTracker tracker;
101 private ReconnectForwarder successor;
103 TransmitQueue(final int targetDepth) {
104 tracker = new AveragingProgressTracker(targetDepth);
107 final Iterable<ConnectionEntry> asIterable() {
108 return Iterables.concat(inflight, pending);
111 final long ticksStalling(final long now) {
112 return tracker.ticksStalling(now);
115 final boolean hasSuccessor() {
116 return successor != null;
119 // If a matching request was found, this will track a task was closed.
120 final Optional<TransmittedConnectionEntry> complete(final ResponseEnvelope<?> envelope, final long now) {
121 Optional<TransmittedConnectionEntry> maybeEntry = findMatchingEntry(inflight, envelope);
122 if (maybeEntry == null) {
123 LOG.debug("Request for {} not found in inflight queue, checking pending queue", envelope);
124 maybeEntry = findMatchingEntry(pending, envelope);
127 if (maybeEntry == null || !maybeEntry.isPresent()) {
128 LOG.warn("No request matching {} found, ignoring response", envelope);
129 return Optional.empty();
132 final TransmittedConnectionEntry entry = maybeEntry.get();
133 tracker.closeTask(now, entry.getEnqueuedTicks(), entry.getTxTicks(), envelope.getExecutionTimeNanos());
135 // We have freed up a slot, try to transmit something
136 int toSend = canTransmitCount(inflight.size());
138 final ConnectionEntry e = pending.poll();
143 LOG.debug("Transmitting entry {}", e);
148 return Optional.of(entry);
152 * Enqueue an entry, possibly also transmitting it.
154 * @return Delay to be forced on the calling thread, in nanoseconds.
156 final long enqueue(final ConnectionEntry entry, final long now) {
157 if (successor != null) {
158 successor.forwardEntry(entry, now);
162 // Reserve an entry before we do anything that can fail
163 final long delay = tracker.openTask(now);
164 if (canTransmitCount(inflight.size()) <= 0) {
165 LOG.trace("Queue is at capacity, delayed sending of request {}", entry.getRequest());
168 // We are not thread-safe and are supposed to be externally-guarded,
169 // hence send-before-record should be fine.
170 // This needs to be revisited if the external guards are lowered.
171 inflight.offer(transmit(entry, now));
172 LOG.debug("Sent request {} on queue {}", entry.getRequest(), this);
178 * Return the number of entries which can be transmitted assuming the supplied in-flight queue size.
180 abstract int canTransmitCount(int inflightSize);
182 abstract TransmittedConnectionEntry transmit(ConnectionEntry entry, long now);
184 final boolean isEmpty() {
185 return inflight.isEmpty() && pending.isEmpty();
188 final ConnectionEntry peek() {
189 final ConnectionEntry ret = inflight.peek();
194 return pending.peek();
197 final void poison(final RequestException cause) {
198 poisonQueue(inflight, cause);
199 poisonQueue(pending, cause);
202 final void setForwarder(final ReconnectForwarder forwarder, final long now) {
203 Verify.verify(successor == null, "Successor {} already set on connection {}", successor, this);
204 successor = Preconditions.checkNotNull(forwarder);
205 LOG.debug("Connection {} superseded by {}, splicing queue", this, successor);
207 ConnectionEntry entry = inflight.poll();
208 while (entry != null) {
209 successor.forwardEntry(entry, now);
210 entry = inflight.poll();
213 entry = pending.poll();
214 while (entry != null) {
215 successor.forwardEntry(entry, now);
216 entry = pending.poll();
221 * We are using tri-state return here to indicate one of three conditions:
222 * - if a matching entry is found, return an Optional containing it
223 * - if a matching entry is not found, but it makes sense to keep looking at other queues, return null
224 * - if a conflicting entry is encountered, indicating we should ignore this request, return an empty Optional
226 @SuppressFBWarnings(value = "NP_OPTIONAL_RETURN_NULL",
227 justification = "Returning null Optional is documented in the API contract.")
228 private static Optional<TransmittedConnectionEntry> findMatchingEntry(final Queue<? extends ConnectionEntry> queue,
229 final ResponseEnvelope<?> envelope) {
230 // Try to find the request in a queue. Responses may legally come back in a different order, hence we need
231 // to use an iterator
232 final Iterator<? extends ConnectionEntry> it = queue.iterator();
233 while (it.hasNext()) {
234 final ConnectionEntry e = it.next();
235 final Request<?, ?> request = e.getRequest();
236 final Response<?, ?> response = envelope.getMessage();
238 // First check for matching target, or move to next entry
239 if (!request.getTarget().equals(response.getTarget())) {
243 // Sanity-check logical sequence, ignore any out-of-order messages
244 if (request.getSequence() != response.getSequence()) {
245 LOG.debug("Expecting sequence {}, ignoring response {}", request.getSequence(), envelope);
246 return Optional.empty();
249 // Check if the entry has (ever) been transmitted
250 if (!(e instanceof TransmittedConnectionEntry)) {
251 return Optional.empty();
254 final TransmittedConnectionEntry te = (TransmittedConnectionEntry) e;
256 // Now check session match
257 if (envelope.getSessionId() != te.getSessionId()) {
258 LOG.debug("Expecting session {}, ignoring response {}", te.getSessionId(), envelope);
259 return Optional.empty();
261 if (envelope.getTxSequence() != te.getTxSequence()) {
262 LOG.warn("Expecting txSequence {}, ignoring response {}", te.getTxSequence(), envelope);
263 return Optional.empty();
266 LOG.debug("Completing request {} with {}", request, envelope);
268 return Optional.of(te);
274 private static void poisonQueue(final Queue<? extends ConnectionEntry> queue, final RequestException cause) {
275 for (ConnectionEntry e : queue) {
276 final Request<?, ?> request = e.getRequest();
277 LOG.trace("Poisoning request {}", request, cause);
278 e.complete(request.toRequestFailure(cause));