zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Powell Molleti (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ZOOKEEPER-901) Redesign of QuorumCnxManager
Date Thu, 30 Mar 2017 04:18:41 GMT

    [ 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.


> 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

View raw message