[ZOOKEEPER-2842] optimize the finish() of Send/RecvWorker in QuorumCnxManager and remove testInitiateConnection() and formates some codes

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Won't Do
• Component/s: quorum
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: Ling Mao
• Created: 12/Jul/17 12:02
• Updated: 05/Jun/19 09:46
• Resolved: 05/Jun/19 09:46

Description

1.the finish() of Send/RecvWorker in QuorumCnxManager changes to double-checked lock style https://en.wikipedia.org/wiki/Double-checked_locking
,a trivial code changes implement a smaller granularity lock to have a better perfermance in too fierce multithread situation.
2.testInitiateConnection() is redundant test function which is only used in TestCase,so I refactor it.
3.some codes don't abide to Java Programme Specification ,so I lift a finger to format them

Issue Links

• links to: GitHub Pull Request #308
• links to: GitHub Pull Request #308