[ZOOKEEPER-2098] QuorumCnxManager: use BufferedOutputStream for initial msg

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.5.0
• Fix Version/s: 3.5.1, 3.6.0
• Component/s: quorum, server
• Assignee: Raúl Gutiérrez Segalés
• Reporter: Raúl Gutiérrez Segalés
• Created: 29/Dec/14 20:34
• Updated: 03/Jun/15 16:46
• Resolved: 29/May/15 21:46

Description

Whilst writing fle-dump (a tool like zk-dump, but to dump FastLeaderElection messages), I noticed that QCM is using DataOutputStream (which doesn't buffer) directly.

So all calls to write() are written immediately to the network, which means simple messaages like two participants exchanging Votes can take a couple RTTs! This is specially terrible for global clusters (i.e.: x-country RTTs).

The solution is to use BufferedOutputStream for the initial negotiation between members of the cluster. Note that there are other places were suboptimal (but not entirely unbuffered) writes to the network still exist. I'll get those in separate tickets.

After using BufferedOutputStream we get only 1 RTT for the initial message, so elections & time for for participants to join a cluster is reduced.

Issue Links

• is related to: ZOOKEEPER-2189 QuorumCnxManager: use BufferedOutputStream for initial msg