[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[GitHub] activemq-artemis pull request #2139: ARTEMIS-1927 treat exceptions and disco...

Github user clebertsuconic commented on a diff in the pull request:
    --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/remoting/server/impl/ ---
    @@ -554,20 +558,13 @@ public void connectionDestroyed(final Object connectionID) {
    - ActiveMQRemoteDisconnectException());
        public void connectionException(final Object connectionID, final ActiveMQException me) {
    -      // We DO NOT call fail on connection exception, otherwise in event of real connection failure, the
    -      // connection will be failed, the session will be closed and won't be able to reconnect
    -      // E.g. if live server fails, then this handler wil be called on backup server for the server
    -      // side replicating connection.
    -      // If the connection fail() is called then the sessions on the backup will get closed.
    -      // Connections should only fail when TTL is exceeded
    +      issueFailure(connectionID, me);
    --- End diff --
    no.. it didn't apply for some time already.
    This is legacy from hornetq where we would only issue disconnects based on TTL.