[QPID-4124] [Java Broker] TransactionTimeout logging has duplication and some erroneously uses default Object.toString

• Type: Bug
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 0.14, 0.15, 0.16, 0.17, 0.18
• Fix Version/s: 0.19
• Component/s: Broker-J
• Assignee: Phil Harvey
• Reporter: Phil Harvey
• Created: 10/Jul/12 13:06
• Updated: 11/Feb/15 20:06
• Resolved: 31/Jul/12 13:29

Description

In AMQChannel, code such as

_logger.warn("IDLE TRANSACTION ALERT " + _logSubject.toString() ...

should be replaced by

_logger.warn("IDLE TRANSACTION ALERT " + _logSubject.toLogString() ...

because the first example uses Object.toString, which is not very useful for logging.