From 6aa10cc1aeb0ffbc6b02bf662b93eab879c517d7 Mon Sep 17 00:00:00 2001 From: Martin Ritchie Date: Mon, 10 Mar 2008 17:16:09 +0000 Subject: QPID-107 : Changes based on code review. git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/branches/M2.1@635602 13f79535-47bb-0310-9956-ffa450edef68 --- java/client/src/main/java/org/apache/qpid/client/AMQSession.java | 4 ++-- .../main/java/org/apache/qpid/client/protocol/AMQProtocolHandler.java | 3 +-- 2 files changed, 3 insertions(+), 4 deletions(-) (limited to 'java/client/src') diff --git a/java/client/src/main/java/org/apache/qpid/client/AMQSession.java b/java/client/src/main/java/org/apache/qpid/client/AMQSession.java index 2ecc0515a5..0a90846b94 100644 --- a/java/client/src/main/java/org/apache/qpid/client/AMQSession.java +++ b/java/client/src/main/java/org/apache/qpid/client/AMQSession.java @@ -2377,7 +2377,7 @@ public class AMQSession extends Closeable implements Session, QueueSession, Topi * @todo Verify the destiation is valid or throw an exception. * @todo Be aware of possible changes to parameter order as versions change. */ - public AMQShortString declareQueue(final AMQDestination amqd, final AMQProtocolHandler protocolHandler) + private AMQShortString declareQueue(final AMQDestination amqd, final AMQProtocolHandler protocolHandler) throws AMQException { /*return new FailoverRetrySupport(*/ @@ -2445,7 +2445,7 @@ public class AMQSession extends Closeable implements Session, QueueSession, Topi return ++_nextProducerId; } - public AMQProtocolHandler getProtocolHandler() + private AMQProtocolHandler getProtocolHandler() { return _connection.getProtocolHandler(); } diff --git a/java/client/src/main/java/org/apache/qpid/client/protocol/AMQProtocolHandler.java b/java/client/src/main/java/org/apache/qpid/client/protocol/AMQProtocolHandler.java index 0faacaabc1..89982a1af0 100644 --- a/java/client/src/main/java/org/apache/qpid/client/protocol/AMQProtocolHandler.java +++ b/java/client/src/main/java/org/apache/qpid/client/protocol/AMQProtocolHandler.java @@ -209,8 +209,7 @@ public class AMQProtocolHandler extends IoHandlerAdapter } catch (RuntimeException e) { - _logger.warn(e.getMessage()); - e.printStackTrace(); + _logger.error(e.getMessage(), e); } if (Boolean.getBoolean("protectio")) -- cgit v1.2.1