ppkarwasz commented on PR #3199:
URL: https://github.com/apache/logging-log4j2/pull/3199#issuecomment-2479880137

   @vy,
   
   Do you have any comments regarding 
[`InternalLoggerRepository`](https://github.com/apache/logging-log4j2/blob/bf79552c902d6638415c78441ddf46771ad5a27b/log4j-core/src/main/java/org/apache/logging/log4j/core/util/internal/InternalLoggerRegistry.java)?
 The class should be similar to the current 
[`LoggerRegistry`](https://github.com/apache/logging-log4j2/blob/2.x/log4j-api/src/main/java/org/apache/logging/log4j/spi/LoggerRegistry.java)
 with two significant changes:
   
   - All unused methods are removed.
   - The order of the keys in inverted: the external map uses `MessageFactory`, 
since there are less message factories than logger names, which reduces the 
number of internal maps.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to