BUG-190 Simplify reconnect logic in protocol-framework.
authorMaros Marsalek <mmarsale@cisco.com>
Wed, 9 Jul 2014 08:05:23 +0000 (10:05 +0200)
committerMaros Marsalek <mmarsale@cisco.com>
Mon, 28 Jul 2014 15:36:59 +0000 (17:36 +0200)
commit44c73b852ad6137c138a90fe0d01d2c82b20b2e5
tree44052f25a044bcbcf6f54ebe57028c71e6476a2a
parent406f0be0d8103297665d8fbc58d2f0db33ad4ebf
BUG-190 Simplify reconnect logic in protocol-framework.

- Removed duplicate code from ReconnectPromise (almost the same code in ProtocolSessionPromise)
- ReconnectPromise now only calls ProtocolSessionPromise.connect (once right away to connect initially,
then every time channel is closed by dropped session)
- Removed second ReconnectingStrategyFactory from ReconnectPromise and AbstractDispatcher.createReconnectingClient
(Kept in AbstractDispatcher for backwards compatibility as deprecated). Only one factory needed for reconnecting client
- Added unit tests to ServerTest to test different cases of reconnecting
- Slightly refactored ProtocolSessionPromise to make code more readable

Change-Id: If3af8f468e7d59822c984cf814d15460ab35921f
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>
pom.xml
src/main/java/org/opendaylight/protocol/framework/AbstractDispatcher.java
src/main/java/org/opendaylight/protocol/framework/ProtocolSessionPromise.java
src/main/java/org/opendaylight/protocol/framework/ReconnectPromise.java
src/main/java/org/opendaylight/protocol/framework/SessionListener.java
src/test/java/org/opendaylight/protocol/framework/ServerTest.java
src/test/java/org/opendaylight/protocol/framework/SimpleDispatcher.java