Return-Path: Delivered-To: apmail-incubator-cassandra-commits-archive@minotaur.apache.org Received: (qmail 18452 invoked from network); 4 Nov 2009 17:45:56 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 4 Nov 2009 17:45:56 -0000 Received: (qmail 32701 invoked by uid 500); 4 Nov 2009 17:45:56 -0000 Delivered-To: apmail-incubator-cassandra-commits-archive@incubator.apache.org Received: (qmail 32665 invoked by uid 500); 4 Nov 2009 17:45:55 -0000 Mailing-List: contact cassandra-commits-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: cassandra-dev@incubator.apache.org Delivered-To: mailing list cassandra-commits@incubator.apache.org Received: (qmail 32655 invoked by uid 99); 4 Nov 2009 17:45:55 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 04 Nov 2009 17:45:55 +0000 X-ASF-Spam-Status: No, hits=-10.5 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_HI X-Spam-Check-By: apache.org Received: from [140.211.11.140] (HELO brutus.apache.org) (140.211.11.140) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 04 Nov 2009 17:45:52 +0000 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id C3644234C04C for ; Wed, 4 Nov 2009 09:45:32 -0800 (PST) Message-ID: <703821268.1257356732786.JavaMail.jira@brutus> Date: Wed, 4 Nov 2009 17:45:32 +0000 (UTC) From: "Gary Dusbabek (JIRA)" To: cassandra-commits@incubator.apache.org Subject: [jira] Updated: (CASSANDRA-488) TcpConnectionManager only ever has one connection In-Reply-To: <1885041595.1255542391414.JavaMail.jira@brutus> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/CASSANDRA-488?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Gary Dusbabek updated CASSANDRA-488: ------------------------------------ Attachment: cassandra-488-v4.patch converted lock to synchronized. > TcpConnectionManager only ever has one connection > ------------------------------------------------- > > Key: CASSANDRA-488 > URL: https://issues.apache.org/jira/browse/CASSANDRA-488 > Project: Cassandra > Issue Type: Improvement > Components: Core > Reporter: Jonathan Ellis > Assignee: Gary Dusbabek > Priority: Minor > Fix For: 0.5 > > Attachments: 488-v3.patch, cassandra-488-v1.patch, cassandra-488-v2.patch, cassandra-488-v4.patch > > > from irc: > jbellis: am i missing something? it looks like TcpConnectionManager.allConnections_ will only ever contain a single TcpConnection, since before adding any connection it checks contains() first, which relies on TcpConn.equals, which reports that any TcpConns w/ same local & remote endpoints are equal > junrao: yes, TcpConnectionManager.allConnections_ can have no more than 1 element. > jbellis: that might explain why it's so easy to swamp things and start getting timeouts -- if all your connections are full of command data, "success" replies will have to wait for those to drain out first before being sent > IMO the Right Design is one connection for commands, one connection for acks/replies, rather than a single pool of connections for everything. that way a connection from A -> B "full" of outgoing commands can't block replies from A -> B acking commands that came the other way. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.