[ 
https://issues.apache.org/jira/browse/ARTEMIS-5110?focusedWorklogId=939533&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-939533
 ]

ASF GitHub Bot logged work on ARTEMIS-5110:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 22/Oct/24 16:07
            Start Date: 22/Oct/24 16:07
    Worklog Time Spent: 10m 
      Work Description: gemmellr commented on code in PR #5303:
URL: https://github.com/apache/activemq-artemis/pull/5303#discussion_r1811014520


##########
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/ActiveMQServerLogger.java:
##########
@@ -1584,8 +1439,6 @@ void 
noQueueIdDefined(org.apache.activemq.artemis.api.core.Message message,
    @LogMessage(id = 224126, value = "Failure during protocol handshake on 
connection to {} from {}", level = LogMessage.Level.ERROR)
    void failureDuringProtocolHandshake(SocketAddress localAddress, 
SocketAddress remoteAddress, Throwable e);
 
-   // notice loggerID=224127 is reserved as it's been used at 
ActiveMQQueueLogger

Review Comment:
   I think its fair to get rid of ActiveMQQueueLogger. I'd just collapse them 
into one, but in a way that didn't require changing the ID or name used, so its 
effectively externally unchanged.





Issue Time Tracking
-------------------

    Worklog Id:     (was: 939533)
    Time Spent: 1.5h  (was: 1h 20m)

> Add ability to identify retired IDs in log annotations
> ------------------------------------------------------
>
>                 Key: ARTEMIS-5110
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-5110
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>            Reporter: Justin Bertram
>            Assignee: Justin Bertram
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 1.5h
>  Remaining Estimate: 0h
>
> The broker uses unique IDs for logging statements. As logging changes over 
> time we need a way to "retire" these IDs (e.g. when certain logging 
> statements are no longer needed) to ensure they are not used again in the 
> future (or perhaps put back into the rotation after a certain amount of time).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information, visit: https://activemq.apache.org/contact


Reply via email to