Hi All: If this is in master, it should go into a JSON module, existing or new but not core. FYI: Dependencies or not, the current JSON layout is in master in log4j-layout-jackson-json. One of the complaints I've heard over the years is that log4j-core is too "big" at 1.6 MB, not to speak of the transitive deps. I worked hard a while back to split out master's core into new modules, so I'd really like to keep the trend going and NOT shove features into the core just because they have no deps.
Gary On Wed, Mar 11, 2020 at 10:09 AM Volkan Yazıcı <[email protected]> wrote: > I have finally removed all dependencies of JsonTemplateLayout, except > for optional JCTools dependency. I have a couple of questions to > proceed further: > > 1. May I move the code in the module to core? > 2. Shall I remove benchmark results in the manual since > it is difficult to keep them up to date? >
