Re: release-2.x EventLogger

2021-03-24 Thread Matt Sicker
Any tests that rely on shared state that can’t be safely multithreaded needs to be categorized as such. I thought I covered everything in API already, but perhaps not. On Wed, Mar 24, 2021 at 21:38 Ralph Goers wrote: > I believe Matt made the tests in the API run in parallel. I’ve often >

Re: release-2.x EventLogger

2021-03-24 Thread Ralph Goers
I believe Matt made the tests in the API run in parallel. I’ve often wondered about that since LogManager is creating contexts meant to be shared across multiple threads. I have no idea how all that will behave in a multi-threaded environment since most unit tests create and destroy

Re: release-2.x EventLogger

2021-03-24 Thread Remko Popma
Wild guess: multi-threading issue? (That's my go-to boogeyman) :-) On Thu, Mar 25, 2021 at 6:03 AM Carter Kozak wrote: > The flake appears to be resolved, just as mysteriously as it appeared. > > -Carter > > On Wed, Mar 24, 2021, at 10:28, Carter Kozak wrote: > > Any ideas why

Re: release-2.x EventLogger

2021-03-24 Thread Gary Gregory
On Wed, Mar 24, 2021 at 5:03 PM Carter Kozak wrote: > > The flake appears to be resolved, just as mysteriously as it appeared. That, or you have special powers ;-) G > > -Carter > > On Wed, Mar 24, 2021, at 10:28, Carter Kozak wrote: > > Any ideas why

Re: release-2.x EventLogger

2021-03-24 Thread Carter Kozak
The flake appears to be resolved, just as mysteriously as it appeared. -Carter On Wed, Mar 24, 2021, at 10:28, Carter Kozak wrote: > Any ideas why org.apache.logging.log4j.EventLoggerTest.structuredData would > fail due to the following? > > java.lang.NullPointerException > > at > >

release-2.x EventLogger

2021-03-24 Thread Carter Kozak
Any ideas why org.apache.logging.log4j.EventLoggerTest.structuredData would fail due to the following? > java.lang.NullPointerException > at > org.apache.logging.log4j.EventLoggerTest.structuredData(EventLoggerTest.java:42) I haven't been able to reproduce this failure locally, but it