Tommy.Tesla

Results 17 comments of Tommy.Tesla

Why do not just add a change event... Above solutions are so tedious.

What's wrong with the CI? The change should have nothing to do with the error. :)

@rgoers Agree! This PR is intended to do that as you proposed. Hope this could be supported ASAP :) . Have a nice day.

@rgoers Do you mean all the pre-loaded Lookups should be loaded via ServiceLoader? I am glad to contribute to this part if you do not mind. :)

@jvz @rgoers It seems the Log4jPlugins class is moved from org.apache.logging.log4j.core.plugins. However in Activator class, Log4jPlugins is still imported as "org.apache.logging.log4j.core.plugins.Log4jPlugins".

@rgoers @jvz Hi, are you busy? Could you get round to take a review and merge :).

@rgoers I propably misunderstood your opinion: **_In essence, StrLookup would become the service interface and every plugin to be pre-loaded would be listed in the services file instead of hard...

@rgoers Hi, rgoers. After debuging and reading the log4j2 codes, I realize that the **_public Interpolator(final Map properties)_** is called several times during factory creation stage. Is that reasonable to...

不好意思同学,最近我们公司在调整,太忙了。 可以使用下面这个包。后续我们会尽快修复正式包 ``` com.github.tbwork MagicGPT 6d39c8fcef ```