[GitHub] [activemq-artemis] clebertsuconic commented on a change in pull request #3777: ARTEMIS-3502 Auto delete & auto create leading to inconsistencies

[GitBox <gi...@apache.org>]

clebertsuconic commented on a change in pull request #3777:
URL: https://github.com/apache/activemq-artemis/pull/3777#discussion_r720197607



##########
File path: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueManagerImpl.java
##########
@@ -34,15 +37,13 @@ private void doIt() {
       Queue queue = server.locateQueue(queueName);
       //the queue may already have been deleted and this is a result of that
       if (queue == null) {
-         if (ActiveMQServerLogger.LOGGER.isDebugEnabled()) {
-            ActiveMQServerLogger.LOGGER.debug("no queue to delete \"" + queueName + "\".");
+         if (logger.isDebugEnabled()) {

Review comment:
       this is logger.debug...
   
   anything related to debug and tracing should be done through an individual logger... Logger.getLogger(className).
   
   this was a mistake




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org