activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From gaohoward <...@git.apache.org>
Subject [GitHub] activemq-artemis pull request #1999: ARTEMIS-1790 Improve Topology Member Fi...
Date Mon, 09 Apr 2018 16:35:17 GMT
Github user gaohoward commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1999#discussion_r180155663
  
    --- Diff: artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/TopologyMemberImpl.java
---
    @@ -105,12 +105,16 @@ public void setUniqueEventID(final long uniqueEventID) {
           return connector;
        }
     
    +   /**
    +    * We only need to check if the connection point to the same node,
    +    * don't need to compare the whole params map.
    +    * @param connection The connection to the target node
    +    * @return true if the connection point to the same node
    +    * as this member represents.
    +    */
        @Override
        public boolean isMember(RemotingConnection connection) {
    -      TransportConfiguration connectorConfig = connection.getTransportConnection() !=
null ? connection.getTransportConnection().getConnectorConfig() : null;
    -
    -      return isMember(connectorConfig);
    -
    +      return connection.isSameTarget(getConnector().getA(), getConnector().getB());
    --- End diff --
    
    The issue here: https://issues.jboss.org/browse/JBEAP-14165
    the issue is that the broker thinks 127.0.0.1 and localhost are different hosts when checking
isMemeber(), where it just compare the configuration parameter map. 


---

Mime
View raw message