Re: Json Template Layout

2020-04-11 Thread Volkan Yazıcı
Hey Ralph, Thanks so much for taking time to review such a lengthy PR, much appreciated. Let me try to address your concerns and share some of mine: Does JTL (i.e., JsonTemplateLayout) support Logstash? I've tried it locally against an actually running Logstash instance and it has worked fine. Th

Re: Build failure in Jenkins build of log4j/release-2.x (10)

2020-04-11 Thread Matt Sicker
Oh I see the issue here. Needs more back ports from master. I’ll update this. On Sat, Apr 11, 2020 at 18:25 Ralph Goers wrote: > Why have these builds been failing for the last week? I looked at it but > whatever is going on is incomprehensible to me. > > Ralph > > > On Apr 11, 2020, at 4:19 PM,

Re: Build failure in Jenkins build of log4j/release-2.x (10)

2020-04-11 Thread Ralph Goers
Why have these builds been failing for the last week? I looked at it but whatever is going on is incomprehensible to me. Ralph > On Apr 11, 2020, at 4:19 PM, Apache Jenkins Server > wrote: > > > There is a build failure in log4j/release-2.x. > > Build: https://builds.apache.org/job/log4j/jo

Json Template Layout

2020-04-11 Thread Ralph Goers
Volkan, I have looked at the PR and the only remaining thing I have an issue with are the changes you made to the logging in the cloud documentation. In my testing, only the Gelf Layout worked properly with an ELK stack. This is because Logstash (as well as Fluentd) are parsing the input trying

[Log4j] Potential simplification to java9 builds?

2020-04-11 Thread Matt Sicker
I was playing around with the pom a little bit yesterday when I came across a relatively new maven-compiler-plugin configuration element called which can be used to output the compiled classes relative to META-INF/versions/N rather than the root. This looks like it would likely remove the need to