vy commented on issue #2766:
URL: 
https://github.com/apache/logging-log4j2/issues/2766#issuecomment-2252813059

   @SnobbyVirus1973, first of all, (as @ppkarwasz indicated) **you are not 
using `ConfigurationBuilder`**, but instead programmatically creating 
components using their builders. This approach we strongly discourage and **you 
should be using `ConfigurationBuilder` instead** – we have documented this 
explicitly in [the staging 
website](https://logging.staged.apache.org/log4j/2.x/manual/customconfig.html), 
which will be available in the next Log4j release.
   
   @SnobbyVirus1973, `NoSqlAppender` doesn't support lookups in the additional 
fields. Hence, what you want is not really possible at this stage _"out of the 
box"_ – that is, you can make it work (by creating your own `NoSqlAppender`, 
etc.), but you really would not want to do that, and it should ideally fixed by 
Log4j.
   
   @ppkarwasz, I am in favor of keeping `PatternLayout` out of the picture and 
instead simply allowing lookups in the additional fields of `NoSqlAppender`. 
Plugging in `PatternLayout` drags along a myriad of other complications.


-- 
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