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 2020/08/17 16:02:15 UTC

[GitHub] [logging-log4j2] rgoers commented on pull request #396: Load pre-loaded lookups via SPI, rather than hard-code in Interpolator.

rgoers commented on pull request #396:
URL: https://github.com/apache/logging-log4j2/pull/396#issuecomment-674967144


   Well, the Interpolator constructor that has the hard-coded list of lookups is initializing the lookups that could be using while constructing the configuration. The other lookups would be used at runtime. There is no reason that I can think of that all the Lookups couldn't be initialized once via ServiceLoader except that it would break compatibility with users who have created their own Lookups as plugins. We could possibly load  the serviceloader Lookups once and then add the ones defined as plugins later.


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