BUG-190 Simplify reconnect logic in protocol-framework. 97/8897/6
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)
- 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>

No differences found