ppkarwasz merged PR #2242:
URL: https://github.com/apache/logging-log4j2/pull/2242
--
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: notifications-unsubsc
ppkarwasz commented on code in PR #2242:
URL: https://github.com/apache/logging-log4j2/pull/2242#discussion_r1466576459
##
log4j-core/src/main/java/org/apache/logging/log4j/core/config/LoggerConfig.java:
##
@@ -649,7 +649,7 @@ public boolean requiresLocation() {
return
ppkarwasz commented on code in PR #2242:
URL: https://github.com/apache/logging-log4j2/pull/2242#discussion_r1466575669
##
log4j-core/src/main/java/org/apache/logging/log4j/core/config/LoggerConfig.java:
##
@@ -614,15 +611,18 @@ public ReliabilityStrategy getReliabilityStrategy(
vy commented on code in PR #2242:
URL: https://github.com/apache/logging-log4j2/pull/2242#discussion_r1466422551
##
log4j-core/src/main/java/org/apache/logging/log4j/core/config/LoggerConfig.java:
##
@@ -614,15 +611,18 @@ public ReliabilityStrategy getReliabilityStrategy() {
ppkarwasz opened a new pull request, #2242:
URL: https://github.com/apache/logging-log4j2/pull/2242
This PR removes all occurences of `instanceof Async*` that might prevent the
separation of the async support.
The removal requires a couple of new API methods and binary incompatible
c