Remove unused class MembershipArithmetic
ISPN-2814 RpcManager doesn't return values properly

* Add proper wrapping of ReplicableCommand return values/exceptions into Response objects, to behave exactly as in case of CacheRpcCommands
* Add unit test using some custom commands to prove result wrapping works identically for CacheRpcCommands and ReplicableCommands in general
* CommandAwareRpcDispatcher checks i the result is already a Response to avoid double wrapping for legacy commands that do their own wrapping internally
ISPN-3033 Lucene index stored in a StringBased CacheStore can get an unaligned bufferSize

There is no test as there was a test already; just that the existing test wouldn't
catch this case because the bufferSize wasn't part of the equals contract: the
assertion used the key.equals to verify correct interpretation.
Fixing the equals activated the unit test: org.infinispan.lucene.Key2StringMapperTest

NoopSegmentReadLockerTest had to be fixed too as there was a mistake which wasn't spotted
as it also took advantage of the same equals flaw.
Move some needlessly verbose logging from debug to trace and fix misspelling
Add a bit of logging and replace viewId with topologyId
RehashAfterJoinWithPreloadTest cleanup
State transfer code cleanup

* Rename StateTransferManager.notifyEndOfTopologyUpdate() to notifyEndOfRebalance() to reflect its actual purpose
* Remove unused method StateTransferManager.isLocalNodeFirst()
* Clarify javadoc LoLocalTopologyManager.confirmRebalance()
ISPN-2589 NPE in InvalidateL1Command

This NPE can appear in both DIST and REPL mode when we process a topology update in which the current node is not a member.

* There is no reason to invalidate segments if we are not a member, so the call to StateConsumerImpl.invalidateSegments() is guarded with a membership check. This also avoids the NPE.
* StateConsumerImpl.invalidateSegments() was wrongly using CommandsFactory.buildInvalidateFromL1Command where it should actually be buildInvalidateCommand()
* Extract all the logic to restart broken transfers from failed sources into a separate method (StateConsumerImpl.restartBrokenTransfers())
* Rename StateConsumerImpl.notifyEndOfTopologyUpdate() to notifyEndOfRebalanceIfNeeded() to clarify its purpose
* Remove StateConsumerImpl.activeTopologyUpdates member as all the logic regarding it has long become redundant
* Remove one of the overloaded variants of CommandsFactory.buildInvalidateFromL1Command that is no longer used