Removes commented out code
Removes unused commit
Removes log statement in NetoworkSender that spammed the log

When attempting to open a channel towards a non listening socket, on
 failure channelClosed() will be called for that channel. However, the
 expected channelOpened() call will not have happened, so NetworkSender
 will not have registered an association between the URI and the Channel.
 channelClosed() will then try to look that up and consider its absense
 an error. That is apparently not correct.
Renames NetworkSender.MessageSender to NetworkSender.NetworkMessageSender

org.neo4j.cluster.com.message.MessageSender kept being imported by
 IntelliJ on any bracked removal, which was annoying. This little rename
 prevents that behaviour.