You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2017/09/07 17:29:00 UTC
[jira] [Commented] (GEODE-3077) Client connection counts should be
accurate
[ https://issues.apache.org/jira/browse/GEODE-3077?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16157285#comment-16157285 ]
ASF subversion and git services commented on GEODE-3077:
--------------------------------------------------------
Commit b92b9a9b0dcaa7f2095d43c02cd9e8141fe45fc8 in geode's branch refs/heads/develop from [~bschuchardt]
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=b92b9a9 ]
GEODE-3555: proper new client protocol closure with more than max conns.
Don't send an old client message when closing new client connections.
This test also tests GEODE-3077.
This closes #756
Signed-off-by: Galen O'Sullivan <go...@pivotal.io>
> Client connection counts should be accurate
> -------------------------------------------
>
> Key: GEODE-3077
> URL: https://issues.apache.org/jira/browse/GEODE-3077
> Project: Geode
> Issue Type: Sub-task
> Components: client/server
> Reporter: Galen O'Sullivan
> Fix For: 1.3.0
>
>
> As a dev user of Geode, I want to be able to look at connection counts and know they are reliable (no matter if I'm using the new client/server protocol or the old). Revisit connection-counting-related methods and ensure the counting mechanism is accurate.
> Specifically:
> Have a look at where and how the following three functions are used:
> {code}
> public void incClientServerCnxCount() {
> this.clientServerCnxCount.incrementAndGet();
> }
> public void decClientServerCnxCount() {
> this.clientServerCnxCount.decrementAndGet();
> }
> public int getClientServerCnxCount() {
> return this.clientServerCnxCount.get();
> }
> {code}
> These are tracking client-server connections. Extract these into common code in {{ServerConnection}} that both {{NewClientServerConnection}} and {{LegacyServerConnection}} can use, and make sure that counts are correct in both creation and cleanup situations. Write some tests to verify.
> Note: client counts look to be tracked separately from connection counts; the old protocol can have multiple connections per client. We're interested with the connection count in this ticket.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)