Bug 2318: Follow-up changes for previous patch 12535

Addressed some comments and other minor changes as a follow-up to patch 12535.

Enhanced the comments in TransactionChainProxy.

Modified Shard to send a failure reply if an runtime exception occurs in
createTransaction. This allows the error to be propagated to the caller
instead of being thrown back to akka.

Modified TransactionChainProxy to throw an IllegalStateException if the
next chain Tx is created before the previous one is readied. Prior, the
Shard would throw an IllegalStateException but from the Future returned
from submit and not on new*Transaction as the API docs state. So this make
it consistent with API contract. This was implemented by introducing a
volatile State field with implemnetations, Idle, Allocated and Closed.

Also, modified TransactionChainProxy to throw TransactionChainClosedException
if the chain is already closed when on Tx create. Again. this makes it
consistent with the API docs.

Added unit tests in DistributedDataStoreIntegrationTest to verify failure
in both cases.

Change-Id: I1b1ed06ceb1d4599f3f6c5443ca24ac1441ac38f
Signed-off-by: tpantelis <tpanteli@brocade.com>

Bug 2337: Fix Tx already sealed failure on Tx commit

Details outlined in bug 2337.

Moved the publishing of the TransactonContext instance in
TransactionFutureCallback#onComplete after the cached operations are
executed.

Also fixed a timing issue in
DistributedDataStoreIntegrationTest#testChangeListenerRegistration that
caused intermittent failures. This was an issue with the test. The
listener registration is done async in the shard so the notification for
the first write commit could occur in the initial notification on
registration which screwed up the test. So I moved the first write
commit before the registration so we expect an initial notification.

Change-Id: Ied2af93be8165208b853c48e57312c3a5acbdaea
Signed-off-by: tpantelis <tpanteli@brocade.com>

Bug 2318: Ensure previous Tx in chain is readied before creating the next

In TransactionChainProxy, subclassed TransactionProxy (ChainedTransactionProxy)
to hook into the TransactionProxy to capture the cohort Futures on ready
and to customize the returned create shard Tx Future.

The previous Tx's ready Futures are captured and cached in the enclosing
TransactionChainProxy instance. When the client creates the next Tx in
the chain, the Future from the shard Tx create message is wrapped by a
sequence Future that combines all the previous ready Futures. Thus, when
the previous ready Futures complete, the shard Tx create is sent and
that Future is used to complete the returned Promise.

Change-Id: If002e4d705510aee3f560c506318d25f386768e5
Signed-off-by: tpantelis <tpanteli@brocade.com>

Introduce a benchmark for databroker

Change-Id: Ica44433b1dd1c863e1c255cc5ec297c56e705fd8
Signed-off-by: Robert Varga <rovarga@cisco.com>

Fix for Bug 2290.

Wrap existing IPv4 code with version test.
Add new IPv6 code with version test.

Change-Id: I30ed4c510dbc79f3e40cdafea4dd46cf37a20f39
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>

Remove l2switch sample

This sample is openflow-specific and creates confusion with the l2switch
project. Remove it.

Change-Id: I03b08bbe08c98c0bd712470c4fb2c29ba65467a3
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bug 2233 - RPC register exception when rpc has no input

* added check that should prevent non routed Rpcs being treated as
routed and and failing at context resolution.

Change-Id: I281152879429f2020f48894753f1d57aea85bde6
Signed-off-by: Jan Hajnar <jhajnar@cisco.com>

Merge "BUG-1953: fix SAL compatility layer"

Merge "Bug 2065 - fixed typo in leaf name"

Fix intermittent failure in DataChangeListenerTest

Fixed an intermittent failure in testDataChangedWithNoSender due to timing if a
DeadLetter is received due to the Monitor message being dropped as the
unit tests don't set up the termination monitor actor. The test expects
no DeadLetter for DataChangedReply so added chanhes to ignore
DeadLetters for other messages.

Change-Id: I39ddc888de7fbff9201ac9f785baa6fec7cb8f2c
Signed-off-by: tpantelis <tpanteli@brocade.com>

BUG 2296 : TransactionProxy should support the ability to accept a local TPC actor path

In Helium when the ShardTransaction processes a Ready message it sends back a the path of a ThreePhaseCommitCohort actor in the ReadyReply. This path is actually a local actor path, this local actor path is then converted into a remote path by the TransactionProxy.

The fix for Bug 1607 breaks this capability which is required to support forward compatibility in a cluster where a transaction request originates in a node that has been upgraded to Helium-1 and the actual transaction is happening on a node which has not yet been upgraded to Helium-1.

Change-Id: I857384bdd61b3492ea270dcf04d14883811c37c2
Signed-off-by: Moiz Raja <moraja@cisco.com>

Merge "Bug 2294: Handle Shard backwards compatibility"

Merge "Bug 2134: Fix intermittent RaftActorTest failure"

BUG 2134 : Make persistence configurable at the datastore level

- Added a peristent flag in the config sub-system
- If persistent in not true then we ignore recovery messages and not use the
  akka persistence apis for persisting anything
- The unit tests
    - assume that persistence is on by default (which it is)
    - test that if recovery is applicable only then the ShardManager and RaftActor process the recovoery messages.
    - test that when persisting the data-persistence API is used in the appropriate places (see RaftActorTest/ShardManagerTest)

Change-Id: I19913bcd32e609ccde6ad8e35209788315504426
Signed-off-by: Moiz Raja <moraja@cisco.com>

Bug 2065 - fixed typo in leaf name

Change-Id: I9f0f201078ffc3d7c08f57108b4d113f87df9ac4
Signed-off-by: Martin Bobak <mbobak@cisco.com>

Bug 2294: Handle Shard backwards compatibility

Implemented as outlined in Bug 2294.

Change-Id: I14aa8ef5f320f9d165492396ece4ea63cce9b0c3
Signed-off-by: tpantelis <tpanteli@brocade.com>

Merge "Bug 2055: Handle Tx create in TransactionProxy resiliently"

Bug 2134: Fix intermittent RaftActorTest failure

We need to wait for akka's recovery to complete before we perform our
tests as akka's recovery runs async and may interfere with our tests and
cause them to fail if the timing is right. This was seen once on
jenkins.

Change-Id: Ia748d17f662d64235881005e6522441b384c78ea
Signed-off-by: tpantelis <tpanteli@brocade.com>

Merge "Cosmetics: check in pom.xml files as _sort_pom_ wants them to be"

Cosmetics: check in pom.xml files as _sort_pom_ wants them to be

Change-Id: I08fc2c0284e0396605cec7d218ea4470a312bfc8
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

Merge "Bug 2265: Modified NormalizedNodeOutputStreamWriter to implement yangtools interface"

Merge "Bug 2252: Terminate ShardWriteTransaction actor on ready"

Merge "BUG 2221 : Add metering to ShardTransaction actor"

Merge "Bug 2086: Adding normalized node  stream reader and writer."

Merge "BUG-2195 Enable features test for netconf connector"

BUG-2195 Enable features test for netconf connector

Change-Id: I85eca361a8af13a87f09477781405dbcb9f721b4
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Bug 2086: Adding normalized node  stream reader and writer.

Normalized node and its children are written in recursive manner. Besides the leafNode, leafSetEntryNode and anyxmlNode, all other nodes must call endNode method
while writing object to data stream. This helps to recreate the object.

Recursion  runs in opposite direction while reading the object.

Updated code with review comments

Change-Id: Ibb822e11fc76f52b5be78596ed979c3b97d51de8
Signed-off-by: Harman Singh <harmasin@cisco.com>

Bug 2055: Handle Tx create in TransactionProxy resiliently

    Modified TransactionProxy to utilize the FindPrimaryShard
functionality
    to wait until the shard is initialized. Also added retries to
    the CreateTransaction operation to wait a period of time for the
shard
    to elect a leader. This makes it more resilient to avoid transient
    timing/sequencing failures, particularly on startup.

    In addition, the FindPrimaryShard and CreateTransaction operations
are now
    done async (non-blocking). The futures are combined to yield the
    CreateTransactionReply which is passed to a new class
    TransactionFutureCallback on completion. The
TransactionFutureCallback
    class creates and stores the TransactionContext which subsequent
    read/write/delete/ready transaction operations can access. If a
    transaction operation occurs before CreateTransaction completes, the
    operation is cached in TransactionFutureCallback and executed when
    the CreateTransaction future completes.

Change-Id: Id9cdc9641038922d6209c44d924bd168658a71fb
Signed-off-by: tpantelis <tpanteli@brocade.com>

Merge "Bug-2098- sal-compatibility not get up-to-date flow information"

Bug 2265: Modified NormalizedNodeOutputStreamWriter to implement yangtools interface

The NormalizedNodeOutputStreamWriter was originally written to implement
a NormalizedNodeStreamWriter interface class that was copied from
yangtools. This was done to get access to the NodeWithValue in the
leafSetEntryNode method which isn't specified in the yangtools
interface. However it turns out we don't actually need the NodeWithValue so
NormalizedNodeOutputStreamWriter was changed to implement the yangtols interface
and the copied NormalizedNodeStreamWriter interface was removed.

Change-Id: I72f0c96359e4fe96e8e183aa7afba9291c5d519d
Signed-off-by: tpantelis <tpanteli@brocade.com>

Bug-2098- sal-compatibility not get up-to-date flow information

When cached is passed in as false, we attempt to collect all the notifications in a guava cache and
return the results after we get the last notification.

Depends on the OFP Patch : https://git.opendaylight.org/gerrit/#/c/11655/

Change-Id: I78d911648957f30d0ae0c92945ad3ac4cf7fc483
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>

Merge "Bug 2055: Handle shard not initialized resiliently"

Merge changes I8b1a0281,Ib6342c74

* changes:
  BUG-1612 Remove ganymed third party ssh lib (replaced by mina ssh)
  BUG-1612 Remove ganymed implementation of SSH server wrapper.

Merge "BUG 1839 - HTTP delete of non existing data"

Merge "Fix checkstyle warnings"

Merge "Convert netconf to use artifacts"

Merge "Import yangtools-artifacts"

Merge "Cleanup RpcRoutingStrategy definition"

BUG-1612 Remove ganymed third party ssh lib (replaced by mina ssh)

Change-Id: I8b1a0281161f29345f1ee1af646a5fca504ad823
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

BUG-1612 Remove ganymed implementation of SSH server wrapper.

Change-Id: Ib6342c74ff9335265d1fe7a12ca8caecbd72d94f
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Merge "BUG-1612 Update netconf testtool to use new ssh server wrapper"

Merge "BUG-1612 Implement mina ssh netconf server endpoint"

Merge "Fix failing reconnect test."

Merge "BUG-2184 Fix subtree filtering for identity-ref leaves"

Merge "BUG-2184 Fix config.yang module(add type as a key for modules list)"

Bug 2210: Fixed initial DCL notification on registration

On RegisterChangeListener message, if the shard isn't the leader, the
listener proxy isn't registered with the IMDS yet, instead a
DelayedListenerRegistration innstance is created and added to a
delayedListenerRegistrations list. When the shard becomes the leader, in
onStateChange, the DelayedListenerRegistration instances in the
delayedListenerRegistrations list are registered with the IMDS.

Change-Id: Id5fbfc3e452e9843f6977139d101e71f4022dc0a
Signed-off-by: tpantelis <tpanteli@brocade.com>

Bug 2252: Terminate ShardWriteTransaction actor on ready

Change-Id: I16f9dcf370aab1e23073163e0e055a40f1415a1a
Signed-off-by: tpantelis <tpanteli@brocade.com>

Convert netconf to use artifacts

Change-Id: I23feabe96c7fc3e0d1ca59b6773abeb97e9467f9
Signed-off-by: Robert Varga <rovarga@cisco.com>

Import yangtools-artifacts

This patch makes use of artifacts exported by yangtools, simplifying the
dependencyManagement section.

Change-Id: Id00030f6f4134c6bebc573cf3c14fe271e59e61a
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Bug-2136 : Clustering : When a transaction is local then do not serialize the Reading/Writing of data"

BUG-1612 Update netconf testtool to use new ssh server wrapper

Change-Id: I8625f8d34df65f6d7e47ad77d393f482afa1abdf
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

BUG-1612 Implement mina ssh netconf server endpoint

Extract sessionCloseCallback and readHandler from AsyncSshHandlerReader

Change-Id: I0e09341fd59f74f7ac57688414a6dac7c9823b8a
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Fix failing reconnect test.

Test verified that session that was dropped during negotiation was not reconnected.
This is no longer true, so the test is failing. However not on jenkins.

Change-Id: I5f416a3397c33607e5004dc48f9569d9da992326
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Merge "[Fix for Bug 1631] - Refactored RaftActorBehavior#handleMessage (and related methods) to return RaftActorBehavior   instead of RaftActorState. - Moved behavior switching from RaftActor based on requested state to the implementations of the   behaviors that should control that switch (Leader/Follower/Candidate) based on the type of   message received. - Pull up logger reference for concrete RAFT behaviors to parent for a common LOG reference. - Updated all logger calls to:   - Use substitution instead of String concatenation.  - Debug logger calls are gated by isDebugEnabled check.  - Correctly use Akka logging API for stack trace on exception.  - Avoid toString() calls during logging.  - Use assigned logger instance (LOG) instead of call to context.getLogger()"

[Fix for Bug 1631]
- Refactored RaftActorBehavior#handleMessage (and related methods) to return RaftActorBehavior
  instead of RaftActorState.
- Moved behavior switching from RaftActor based on requested state to the implementations of the
  behaviors that should control that switch (Leader/Follower/Candidate) based on the type of
  message received.
- Pull up logger reference for concrete RAFT behaviors to parent for a common LOG reference.
- Updated all logger calls to:
  - Use substitution instead of String concatenation.
- Debug logger calls are gated by isDebugEnabled check.
- Correctly use Akka logging API for stack trace on exception.
- Avoid toString() calls during logging.
- Use assigned logger instance (LOG) instead of call to context.getLogger()

Change-Id: I376e21734a0540aae714fe9ef1562d74e73e558b
Signed-off-by: mark.mozolewski <mark.mozolewski@hp.com>

Bug 2002: Classpath error when loading neutron northbound api

Add the org.eclipse.persistence.antlr to the odl-pseristence feture. There are weird timing issues where if the antlr bundle is not loaded before core and moxy then the classes in antlr cannot be found.

Change-Id: I8fd30b5fb215b8a25512deac45e80099c91c539e
Signed-off-by: Sam Hague <shague@redhat.com>
(cherry picked from commit 3ee4966ad2d840a907fae33ff74037d599765742)

BUG-2184 Fix subtree filtering for identity-ref leaves

Change-Id: I7d367c793a461ae26f56ddf611f7e45a723f28fc
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

BUG-2184 Fix config.yang module(add type as a key for modules list)

Netconf server requires the type and name when e.g. deleting data so the yang model did not fully reflect the implementation

Change-Id: If4b04fe1ec427cff0f52d743cc1440fc7fe143d2
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Merge "BUG-2207 Make reconnect promise reconnect even if session was dropped during negotiation"

Merge "BUG-2217 Add missing namespace to serialized inner runtime beans"

BUG-2207 Make reconnect promise reconnect even if session was dropped during negotiation

Change-Id: I58adc2e42bb8ce09bfb8b5cd214a91300bb2bbb0
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

BUG 1833 : Remove all unused code from sal-clustering-commons

Change-Id: I7ce05b4ee77b6151f361091b18d1a7bbd8ed408a
Signed-off-by: Moiz Raja <moraja@cisco.com>
(cherry picked from commit 4861f2406206bf208a07a703ad8d6808072c6e45)

Merge "Bug2174: XSQL log file is hardcode to /tmp/xql.log"

BUG 2221 : Add metering to ShardTransaction actor

Change-Id: I4cbf74f9e8979b996a30a019657177b92fbdfc18
Signed-off-by: Abhishek Kumar <abhishk2@cisco.com>
Signed-off-by: Moiz Raja <moraja@cisco.com>

Merge "Add support for metadata to the Match/Action classes"

BUG-2217 Add missing namespace to serialized inner runtime beans

Change-Id: I267c23ec620bacbd800ee171f37413d4f5997f69
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Merge "Add get-config commit edit-config to testtool"

Merge "Issue fix for config subsystem"

Add get-config commit edit-config to testtool

Change-Id: Ie85c839c2077b9d7ca560dc42120268501acb8b0
Signed-off-by: Andrej Socha <ansocha@cisco.com>
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Bug2174: XSQL log file is hardcode to /tmp/xql.log

Change XSQL to point to java.io.tmpdir which should be data\tmp\xql.log

Change-Id: I811064dd28ce2f4cdb661c7bf968a1cee21d32be
Signed-off-by: Christopher O'Shea <christopher.o.shea@ericsson.com>

BUG 1839 - HTTP delete of non existing data

Before - after deleting non existing data an exception was raised.
Now - after deleting non existing data it is checked whether there is in
exception chain instance of ModifiedNodeDoesNotExistException. If it is so then
HTTP 404 - data missing status is returned.

Change-Id: I43ccaa16abe7dfaffa3e031aa31d93f8f106a81f
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>

Issue fix for config subsystem

Change-Id: I73d02158accb65c92dabc76eb8b353b052ac2d05
Signed-off-by: Filip Tehlar <ftehlar@cisco.com>

Merge "Bug 1577: Gates access to Shard actor until its initialized"

Merge "Fixes bug 2114"

Add support for metadata to the Match/Action classes

- Allow to attach some property to Match and Action so they can be
used for passing extra informations, that applications can use.

Change-Id: I4225271d7cd3429b11adb191ae178f541bbd635d
Signed-off-by: Giovanni Meo <gmeo@cisco.com>

Merge "BUG 720 - YANG leaf as JSON input *<*:* couldn't be saved"

BUG 720 - YANG leaf as JSON input *<*:* couldn't be saved

If input value for leaf in JSON format contained opending angle bracket
(<) followed by collon (:) then exception was raised.

Because schema isn't present at time of JSON input reading it was also
supposed that data could be of type leafref which is specified in format
module:value. Module part is then transformed to instance of URI object.

Chracter < isn't permitted in URI according RFC2396 for URI and while
creating instance of URI class the exception which wasn't caught was
raised and transformed to returning error message.

Change-Id: I46bb949b38623f62a02daf4390c373371775ae1f
Signed-off-by: Jozef Gloncak <jgloncak@cisco.com>

Fix checkstyle warnings

Reorder imports and rename logger->LOG.

Change-Id: I75929957f0ec6d4f70d274c86f4de3a92c248a9a
Signed-off-by: Robert Varga <rovarga@cisco.com>

Cleanup RpcRoutingStrategy definition

Adds preconditions and final keywords as appropriate, removes single-use
private static constructors.

Change-Id: If3bc7c46cb46edb1e2059453f42e1bfc521af46d
Signed-off-by: Robert Varga <rovarga@cisco.com>

Fix compilation after jacoco version bump

The plugin configuration for 0.6.3+ has been changed incompatibly, so
fix it up.

Change-Id: Ia8e96f9a98e18810680e18f9bd59cbb54873c331
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bug 2055: Handle shard not initialized resiliently

Added a flag to the FindLocalShard and FindPrimaryShard messages to
specify whether or not to wait for he shard to be initialized.

Modified FindLocalShard and FindPrimaryShard message handling in the
ShardManager to wait for the shard to be initialized before replying if
the flag is set in the message. This is done async by caching the
response info in the ShardInformation and sending the response when the
ActorInitialized message for the shard is received.

Modified DistributedDataStore#registerChangeListener to always create a
DataChangeListenerRegistrationProxy and moved the code that performs the
findLocalShard and RegisterChangeListener operations into
DataChangeListenerRegistrationProxy. In DataChangeListenerRegistrationProxy,
the findLocalShard operation is done async (with a long time out). On success,
the RegisterChangeListener message is sent.

Added change listener test to DistributedDataStoreIntegrationTest and
removed the DistributedDataStoreTest since most of the tests tested
registration and the bulk of that logic is now in
DataChangeListenerRegistrationProxy with associated tests. The rest of
the tests in DistributedDataStoreTest are covered by
DistributedDataStoreIntegrationTest.

Change-Id: I9f0d412801110c97eb48ecc4d0fd77ee081a7e81
Signed-off-by: tpantelis <tpanteli@brocade.com>

Bug-2136 : Clustering : When a transaction is local then do not serialize the Reading/Writing of data

Ensured that the TransactionContext created for each shard within the transaction has a flag isShardLocal.
if the flag is false, we would be using ProfoBuf to serialize.

Change-Id: I3dc754ae58566212237a6ce385e5f5bb63afd4e0
Signed-off-by: Kamal Rameshan <kramesha@cisco.com>

Merge "Update the XSQL jdbc driver to support pentaho"

Adding restconf to the opendaylight-karaf distribution

Change-Id: I91cbee656e2f4da2af62f8a26e7800401f9c6aa4
Signed-off-by: Colin Dixon <colin@colindixon.com>

Merge "Bug 2003: CDS serialization improvements"

Bug 2135: Create ShardInformation on startup

Modified the ShardManager to create the ShardInformation map on startup,
but without creating the shard actors yet, instead of when the SchemaContext is initialized.

When the SchemaContext is fully initialized then create the shard actors.

On FindLocalShard and FindPrimaryShard messages, only return a valid
response if the shard actor is created and it's initialized.

Change-Id: I361f2af9e53878f62c5890350afd7a5f2877b95c
Signed-off-by: tpantelis <tpanteli@brocade.com>

Merge "Bug 2160: Added concurrent 3-phase commit coordinator"

Merge "Bug 865: Fixed use of removed deprecated YANGInstanceIdentifier methods."

Merge "Bug 2038: Ensure only one concurrent 3-phase commit in Shard"

Bug 865: Fixed use of removed deprecated YANGInstanceIdentifier methods.

Some components in controller still used methods which was deprecated
before Helium and was removed after Helium was released to accomodate
API changes for 0.7.0 Yangtools. This patch migrates code to replacement
methods.

Change-Id: Ia7ef044eb3af4f5652dc71d9cba2bb64efd95187
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Bug 865: Fixed use of removed deprecated YANGInstanceIdentifier methods."

Bug 865: Fixed use of removed deprecated YANGInstanceIdentifier methods.

Some components in controller still used methods which was deprecated
before Helium and was removed after Helium was released to accomodate
API changes for 0.7.0 Yangtools. This patch migrates code to replacement
methods.

Change-Id: I146a08caefeb0d666223cf66600a4342a5b49165
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Merge "Migrate netconf testtool to work with karaf distribution"

Merge "Bug 1764 - added service for pushing default node configuration"

Merge "Add features-netconf-connector to controller karaf distro"

Merge "BUG-2006: Rework state tracking"

Migrate netconf testtool to work with karaf distribution

Make testtool part of default build process.

Change-Id: Ie957bcf2a38405fee979a6f8547b328d3bd8e2ea
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Add features-netconf-connector to controller karaf distro

Change-Id: I44bf8d82f35586a03950d5ef2f26a321931cb8c6
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Bug 2038: Ensure only one concurrent 3-phase commit in Shard

Added a ShardCommitCoordinator class that ensures there's only one
concurrent 3-phase commit.

The following outlines the new commit workflow:

- On ready, the ShardTransaction creates the dom store cohort and
forwards a new ForwardedReadyTransaction message to the shard.

- The shard calls its ShardCommitCoordinator to add the cohort and
modificaton to a cached keyed by transaction ID.

- On CanCommitTransaction message, the ShardCommitCoordinator looks up and removes
the cohort entry from the cache corresponding to the transaction ID
passed via the CanCommit message. The ShardCommitCoordinator also caches
the cohort entry for the current transaction in progress. If there's no
transaction in progress, the committing transaction becomes the current
transaction and canCommit is called on the cohort. Otherwise, the cohort
entry is queued to be processed after the current tranaction completes.

- On CommitTransaction message, if the transaction ID passed via the
Commit message matches the currently cached cohort entry, the preCommit
and commit phases are performed. When complete, the ShardCommitCoordinator
dequeues the next waiting transaction cohort entry, if any, and process
it.

If a Tx is aborted and it is the current transaction, the
ShardCommitCoordinator handles it as a completed Tx.

Implemented a timeout mechanism using the akka scheduler such that if
the commit message isn't received after a period of time (default 30 s)
after the canCommit message, the transaction is aborted so that the next
transaction can proceed. This is to handle remote node or network
failures during a 3-phrase commit.

The ThreePhaseCommitCohort actor was removed along with the
ForwardedCommitTransaction.

Change-Id: Iaa5692ca45cd7635d1a06a609f4bf98bec50df14
Signed-off-by: tpantelis <tpanteli@brocade.com>

Merge "Bug 1965: Fixed DataChangedReply sent to deadletters"