[jira] [Commented] (ZOOKEEPER-901) Redesign of QuorumCnxManager

["Powell Molleti (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ZOOKEEPER-901?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15948402#comment-15948402 ] 

Powell Molleti commented on ZOOKEEPER-901:
------------------------------------------

I managed to create a PR ( https://github.com/geek101/zookeeper/pull/6 ) against trunk out of the work I have so far. Once there is some closure on ZOOKEEPER-236 and ZOOKEEPER-2184 I will be able to make more progress to get an official PR for these changes.

Thanks
Powell.

> Redesign of QuorumCnxManager
> ----------------------------
>
>                 Key: ZOOKEEPER-901
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-901
>             Project: ZooKeeper
>          Issue Type: Improvement
>          Components: leaderElection
>    Affects Versions: 3.3.1
>            Reporter: Flavio Junqueira
>            Assignee: Michael Han
>             Fix For: 3.6.0
>
>
> QuorumCnxManager manages TCP connections between ZooKeeper servers for leader election in replicated mode. We have identified over time a couple of deficiencies that we would like to fix. Unfortunately, fixing these issues requires a little more than just generating a couple of small patches. More specifically, I propose, based on previous discussions with the community, that we reimplement QuorumCnxManager so that we achieve the following:
> # Establishing connections should not be a blocking operation, and perhaps even more important, it shouldn't prevent the establishment of connections with other servers;
> # Using a pair of threads per connection is a little messy, and we have seen issues over time due to the creation and destruction of such threads. A more reasonable approach is to have a single thread and a selector.  



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)