*/
package org.opendaylight.protocol.bgp.rib.impl.protocol;
-import com.google.common.base.Preconditions;
+import static com.google.common.base.Preconditions.checkState;
+import static java.util.Objects.requireNonNull;
+
import io.netty.bootstrap.Bootstrap;
import io.netty.channel.ChannelFuture;
import io.netty.channel.ChannelFutureListener;
import io.netty.util.concurrent.Promise;
import java.net.InetSocketAddress;
import java.util.concurrent.TimeUnit;
-import javax.annotation.Nonnull;
-import javax.annotation.concurrent.GuardedBy;
+import org.checkerframework.checker.lock.qual.GuardedBy;
+import org.eclipse.jdt.annotation.NonNull;
import org.opendaylight.protocol.bgp.rib.impl.StrictBGPPeerRegistry;
import org.opendaylight.protocol.bgp.rib.impl.spi.BGPPeerRegistry;
import org.opendaylight.protocol.bgp.rib.impl.spi.PeerRegistrySessionListener;
import org.opendaylight.protocol.bgp.rib.spi.BGPSession;
-import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.IpAddress;
+import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.IpAddressNoZone;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
public final class BGPProtocolSessionPromise<S extends BGPSession> extends DefaultPromise<S> {
private static final Logger LOG = LoggerFactory.getLogger(BGPProtocolSessionPromise.class);
private static final int CONNECT_TIMEOUT = 5000;
-
- private InetSocketAddress address;
private final int retryTimer;
private final Bootstrap bootstrap;
- private final BGPPeerRegistry peerRegistry;
@GuardedBy("this")
private final AutoCloseable listenerRegistration;
@GuardedBy("this")
+ private InetSocketAddress address;
+ @GuardedBy("this")
private ChannelFuture pending;
@GuardedBy("this")
private boolean peerSessionPresent;
private boolean connectSkipped;
- public BGPProtocolSessionPromise(@Nonnull final InetSocketAddress remoteAddress, final int retryTimer,
- @Nonnull final Bootstrap bootstrap, @Nonnull final BGPPeerRegistry peerRegistry) {
+ public BGPProtocolSessionPromise(final @NonNull InetSocketAddress remoteAddress, final int retryTimer,
+ final @NonNull Bootstrap bootstrap, final @NonNull BGPPeerRegistry peerRegistry) {
super(GlobalEventExecutor.INSTANCE);
- this.address = Preconditions.checkNotNull(remoteAddress);
+ address = requireNonNull(remoteAddress);
this.retryTimer = retryTimer;
- this.bootstrap = Preconditions.checkNotNull(bootstrap);
- this.peerRegistry = Preconditions.checkNotNull(peerRegistry);
- this.listenerRegistration = this.peerRegistry.registerPeerSessionListener(
- new BGPProtocolSessionPromise.PeerRegistrySessionListenerImpl(this,
- StrictBGPPeerRegistry.getIpAddress(this.address)));
+ this.bootstrap = requireNonNull(bootstrap);
+ listenerRegistration = requireNonNull(peerRegistry).registerPeerSessionListener(
+ new PeerRegistrySessionListenerImpl(StrictBGPPeerRegistry.getIpAddress(address)));
}
+ @SuppressWarnings("checkstyle:illegalCatch")
public synchronized void connect() {
- if (this.peerSessionPresent) {
- LOG.debug("Connection to {} already exists", this.address);
- this.connectSkipped = true;
+ if (peerSessionPresent) {
+ LOG.debug("Connection to {} already exists", address);
+ connectSkipped = true;
return;
- } else {
- this.connectSkipped = false;
}
- final BGPProtocolSessionPromise lock = this;
+ connectSkipped = false;
+
+ final BGPProtocolSessionPromise<?> lock = this;
try {
- LOG.debug("Promise {} attempting connect for {}ms", lock, Integer.valueOf(CONNECT_TIMEOUT));
- if (this.address.isUnresolved()) {
- this.address = new InetSocketAddress(this.address.getHostName(), this.address.getPort());
+ LOG.debug("Promise {} attempting connect for {}ms", lock, CONNECT_TIMEOUT);
+ if (address.isUnresolved()) {
+ address = new InetSocketAddress(address.getHostName(), address.getPort());
}
- this.bootstrap.option(ChannelOption.CONNECT_TIMEOUT_MILLIS, CONNECT_TIMEOUT);
- this.bootstrap.remoteAddress(this.address);
- final ChannelFuture connectFuture = this.bootstrap.connect();
- connectFuture.addListener(new BGPProtocolSessionPromise.BootstrapConnectListener(lock));
- this.pending = connectFuture;
+ bootstrap.option(ChannelOption.CONNECT_TIMEOUT_MILLIS, CONNECT_TIMEOUT);
+ bootstrap.remoteAddress(address);
+ final ChannelFuture connectFuture = bootstrap.connect();
+ connectFuture.addListener(new BootstrapConnectListener());
+ pending = connectFuture;
} catch (final Exception e) {
- LOG.info("Failed to connect to {}", this.address, e);
- this.setFailure(e);
+ LOG.warn("Failed to connect to {}", address, e);
+ setFailure(e);
}
}
- public synchronized void reconnect() {
- if (this.retryTimer == 0) {
+ synchronized void reconnect() {
+ if (retryTimer == 0) {
LOG.debug("Retry timer value is 0. Reconnection will not be attempted");
- this.setFailure(this.pending.cause());
+ setFailure(pending.cause());
return;
}
- final BGPProtocolSessionPromise lock = this;
- final EventLoop loop = this.pending.channel().eventLoop();
+ final EventLoop loop = pending.channel().eventLoop();
loop.schedule(() -> {
synchronized (BGPProtocolSessionPromise.this) {
if (BGPProtocolSessionPromise.this.peerSessionPresent) {
LOG.debug("Connection to {} already exists", BGPProtocolSessionPromise.this.address);
BGPProtocolSessionPromise.this.connectSkipped = true;
return;
- } else {
- BGPProtocolSessionPromise.this.connectSkipped = false;
}
+
+ BGPProtocolSessionPromise.this.connectSkipped = false;
LOG.debug("Attempting to connect to {}", BGPProtocolSessionPromise.this.address);
final ChannelFuture reconnectFuture = BGPProtocolSessionPromise.this.bootstrap.connect();
- reconnectFuture.addListener(new BootstrapConnectListener(lock));
+ reconnectFuture.addListener(new BootstrapConnectListener());
BGPProtocolSessionPromise.this.pending = reconnectFuture;
}
- }, this.retryTimer, TimeUnit.SECONDS);
- LOG.debug("Next reconnection attempt in {}s", this.retryTimer);
+ }, retryTimer, TimeUnit.SECONDS);
+ LOG.debug("Next reconnection attempt in {}s", retryTimer);
}
@Override
public synchronized boolean cancel(final boolean mayInterruptIfRunning) {
closePeerSessionListener();
if (super.cancel(mayInterruptIfRunning)) {
- Preconditions.checkNotNull(this.pending);
- this.pending.cancel(mayInterruptIfRunning);
+ requireNonNull(pending);
+ pending.cancel(mayInterruptIfRunning);
return true;
- } else {
- return false;
}
+
+ return false;
}
+ @SuppressWarnings("checkstyle:illegalCatch")
private synchronized void closePeerSessionListener() {
try {
- this.listenerRegistration.close();
+ listenerRegistration.close();
} catch (final Exception e) {
LOG.debug("Exception encountered while closing peer registry session listener registration", e);
}
return super.setSuccess(result);
}
- private class BootstrapConnectListener implements ChannelFutureListener {
- @GuardedBy("this")
- private final Object lock;
-
- BootstrapConnectListener(final Object lock) {
- this.lock = lock;
- }
-
+ private final class BootstrapConnectListener implements ChannelFutureListener {
@Override
public void operationComplete(final ChannelFuture channelFuture) throws Exception {
- synchronized (this.lock) {
- BGPProtocolSessionPromise.LOG.debug("Promise {} connection resolved", this.lock);
- Preconditions.checkState(BGPProtocolSessionPromise.this.pending.equals(channelFuture));
- if (BGPProtocolSessionPromise.this.isCancelled()) {
+ synchronized (BGPProtocolSessionPromise.this) {
+ LOG.debug("Promise {} connection resolved", BGPProtocolSessionPromise.this);
+ checkState(BGPProtocolSessionPromise.this.pending.equals(channelFuture), "Unexpected promise %s",
+ channelFuture);
+ if (isCancelled()) {
if (channelFuture.isSuccess()) {
- BGPProtocolSessionPromise.LOG.debug("Closing channel for cancelled promise {}", this.lock);
+ LOG.debug("Closing channel for cancelled promise {}", BGPProtocolSessionPromise.this);
channelFuture.channel().close();
}
} else if (channelFuture.isSuccess()) {
- BGPProtocolSessionPromise.LOG.debug("Promise {} connection successful", this.lock);
+ LOG.debug("Promise {} connection successful", BGPProtocolSessionPromise.this);
} else {
- BGPProtocolSessionPromise.LOG.debug("Attempt to connect to {} failed", BGPProtocolSessionPromise.this.address, channelFuture.cause());
+ LOG.warn("Attempt to connect to {} failed", BGPProtocolSessionPromise.this.address,
+ channelFuture.cause());
BGPProtocolSessionPromise.this.reconnect();
}
}
}
private class PeerRegistrySessionListenerImpl implements PeerRegistrySessionListener {
- @GuardedBy("this")
- private final Object lock;
- private final IpAddress peerAddress;
+ private final IpAddressNoZone peerAddress;
- PeerRegistrySessionListenerImpl(final Object lock, final IpAddress peerAddress) {
- this.lock = lock;
+ PeerRegistrySessionListenerImpl(final IpAddressNoZone peerAddress) {
this.peerAddress = peerAddress;
}
@Override
- public void onSessionCreated(@Nonnull final IpAddress ip) {
- if (!ip.equals(this.peerAddress)) {
- return;
- }
- BGPProtocolSessionPromise.LOG.debug("Callback for session creation with peer {} received", ip);
- synchronized (this.lock) {
- BGPProtocolSessionPromise.this.peerSessionPresent = true;
+ public void onSessionCreated(final IpAddressNoZone ip) {
+ if (ip.equals(peerAddress)) {
+ LOG.debug("Callback for session creation with peer {} received", ip);
+ synchronized (BGPProtocolSessionPromise.this) {
+ BGPProtocolSessionPromise.this.peerSessionPresent = true;
+ }
}
}
@Override
- public void onSessionRemoved(@Nonnull final IpAddress ip) {
- if (!ip.equals(this.peerAddress)) {
- return;
- }
- BGPProtocolSessionPromise.LOG.debug("Callback for session removal with peer {} received", ip);
- synchronized (this.lock) {
- BGPProtocolSessionPromise.this.peerSessionPresent = false;
- if (BGPProtocolSessionPromise.this.connectSkipped) {
- BGPProtocolSessionPromise.this.connect();
+ public void onSessionRemoved(final IpAddressNoZone ip) {
+ if (ip.equals(peerAddress)) {
+ LOG.debug("Callback for session removal with peer {} received", ip);
+ synchronized (BGPProtocolSessionPromise.this) {
+ BGPProtocolSessionPromise.this.peerSessionPresent = false;
+ if (BGPProtocolSessionPromise.this.connectSkipped) {
+ BGPProtocolSessionPromise.this.connect();
+ }
}
}
}
}
-
}
Code Review / bgpcep.git / blobdiff