Mailing-List: contact issues-help@geode.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@geode.apache.org
Date: Tue, 6 Dec 2016 19:17:58 +0000 (UTC)
From: "Dan Smith (JIRA)" <jira@apache.org>
To: issues@geode.incubator.apache.org
Message-ID: <JIRA.13025254.1480722420000.453494.1481051878390@Atlassian.JIRA>
In-Reply-To: <JIRA.13025254.1480722420000@Atlassian.JIRA>
References: <JIRA.13025254.1480722420000@Atlassian.JIRA> <JIRA.13025254.1480722420920@arcas>
Subject: [jira] [Commented] (GEODE-2179) GatewaySenderMBeans isConnected
 flag does not notice if a connection is destroyed
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Tue, 06 Dec 2016 19:18:02 -0000


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

Dan Smith commented on GEODE-2179:
----------------------------------

Hi Amey,

My mistake, I didn't assign this one to myself. But I actually have a review request out for a fix for this issue - let me know what you think:

https://reviews.apache.org/r/54330/

> GatewaySenderMBeans isConnected flag does not notice if a connection is destroyed
> ---------------------------------------------------------------------------------
>
>                 Key: GEODE-2179
>                 URL: https://issues.apache.org/jira/browse/GEODE-2179
>             Project: Geode
>          Issue Type: Bug
>          Components: wan
>            Reporter: Dan Smith
>            Assignee: Amey Barve
>
> The GatewaySenderMBean has a method called isConnected which ultimately calls this code in GatewaySenderEventRemoteDispatcher
> {code}
> @Override
> public boolean isConnectedToRemote() {
>   return connection != null;
> }
> {code}
> However, this doesn't take into account that the connection may be destroyed, but not null. Therefore the mbean can incorrectly indicate that the sender is connected.


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)