You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@logging.apache.org by GitBox <gi...@apache.org> on 2019/05/22 02:53:49 UTC

[GitHub] [logging-log4j2] remkop commented on issue #272: LOG4J2-2611: AsyncQueueFullPolicy configuration short values are case insensitive

remkop commented on issue #272: LOG4J2-2611: AsyncQueueFullPolicy configuration short values are case insensitive
URL: https://github.com/apache/logging-log4j2/pull/272#issuecomment-494632334
 
 
   I reviewed this PR to make configuration case-insensitive and improve the javadoc by adding a warning for the SYNCHRONOUS event route.
   
   Looks good to merge.
   
   (As an aside, I've started to dislike the "SYNCHRONOUS" name for that event route. It sounds too innocuous. :-) It should really be called something like "BYPASS_QUEUE" or something... Anyway, a bit late to change that perhaps.)

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services