Github user franz1981 commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/1849#discussion_r165944739
--- Diff:
artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/OpenWireConnection.java
---
@@ -713,12 +730,16 @@ public void reconnect(AMQConnectionContext
existingContext, ConnectionInfo info)
context.incRefCount();
}
+ private static void traceSendCommand(Command command) {
+ ActiveMQServerLogger.LOGGER.trace("sending " + command);
+ }
+
/**
* This will answer with commands to the client
*/
public boolean sendCommand(final Command command) {
- if (ActiveMQServerLogger.LOGGER.isTraceEnabled()) {
- ActiveMQServerLogger.LOGGER.trace("sending " + command);
+ if (ACTIVE_MQ_SERVER_LOGGER_TRACE_ENABLED) {
--- End diff --
I know and it seems an issue with this logger, because it creates a point
of contention between all the threads (all in the broker!) because it is locked
(I will provide soon a trace of it): I don't think could exist a clean way to
fix it other than on the jboss logger and it isn't feasible...
---
