[
https://issues.apache.org/jira/browse/ARTEMIS-501?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15256325#comment-15256325
]
ASF GitHub Bot commented on ARTEMIS-501:
----------------------------------------
Github user asfgit closed the pull request at:
https://github.com/apache/activemq-artemis/pull/483
> Incorrect log message in class ClientSessionFactoryImpl
> -------------------------------------------------------
>
> Key: ARTEMIS-501
> URL: https://issues.apache.org/jira/browse/ARTEMIS-501
> Project: ActiveMQ Artemis
> Issue Type: Bug
> Components: AMQP
> Affects Versions: 1.2.0
> Reporter: Chen Maoqian
> Fix For: 1.2.0
>
>
> There is incorrect logging message present in method establishNewConnection()
> in class
> org.apache.activemq.artemis.core.client.impl.ClientSessionFactoryImpl .
> protected RemotingConnection establishNewConnection() {
> ....
> RemotingConnection newConnection = clientProtocolManager.connect(...);
> newConnection.addFailureListener(new
> DelegatingFailureListener(newConnection.getID()));
> schedulePing();
> if (ActiveMQClientLogger.LOGGER.isTraceEnabled()) {
> ActiveMQClientLogger.LOGGER.trace("returning " + connection);
> }
> return newConnection;
> }
> Log message says, that it is returning connection (old one, present as an
> attribute of class), but connection stored in newConnection variable is
> returned (and then it is set as this.connection).
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
