Bug 1688: Fix IllegalStateEx from FlowCapableTopologyExporter

Modified code to enqueue the deletes on the OperationProcessor in
removeAffectedLinks.

Also added unit tests..

Change-Id: Ie7764303e6c18ff020567739e17e51aed76a8762
Signed-off-by: tpantelis <tpanteli@brocade.com>
Bug 1819 - Moved bundle up in features.xml to avoid exception in log
 
Change-Id: Icbd3d8957d38ca74797c11f2c60683fa017c3246
Signed-off-by: Devin Avery <devin.avery@brocade.com>
BUG 1883 - Ensure that all debug logging is done conditionally

This commit is mostly about conditional logging.

It also sneaks in a couple of other things,
1. Cleanup of ClientRequestTracker
2. Removal of some dead code

Change-Id: I0862f1273e94856e19107d3a4beec4d66452787d
Signed-off-by: Moiz Raja <moraja@cisco.com>
Signed-off-by: Harman Singh <harmasin@cisco.com>
Merge "Bug 1688: Fix IllegalStateEx from FlowCapableTopologyExporter"
Merge "BUG 1883 - Ensure that all debug logging is done conditionally"
Merge "Bug 1819 - Moved bundle up in features.xml to avoid exception in log   Change-Id: Icbd3d8957d38ca74797c11f2c60683fa017c3246 Signed-off-by: Devin Avery <devin.avery@brocade.com>"