Re: Location performance

2019-07-02 Thread Ralph Goers
Well, I just noticed that GcFreeSynchronousLoggingTest is generating a bunch of errors. It seems some of the patterns in gcFreeLogging.xml have location info in the patterns so StackTraceElements are being created. It would probably be simpler to honor the flag then try to reconcile the issues.

Re: Location performance

2019-07-02 Thread Ralph Goers
Never answer email from the phone. It spell checks everything wrong. Ralph > On Jul 2, 2019, at 3:54 PM, Ralph Goers wrote: > > I am using these because the reporter provided them. I will convert them to > John at some point. > > Log4J 3.0 is the master branch. Since it uses java 8 some thing

Re: Location performance

2019-07-02 Thread Ralph Goers
I am using these because the reporter provided them. I will convert them to John at some point. Log4J 3.0 is the master branch. Since it uses java 8 some things are easier to do than on release-2.x. The withLocation test uses the new builder I add to the api. The witHLocation method doesn’t hav

Re: Location performance

2019-07-02 Thread Remko Popma
If memory serves correctly, the `includeLocation` was introduced because it was necessary for async loggers (and it seemed not feasible to get this information from the layout pattern). Async loggers need to know upfront whether they need to pass location information to the consumer thread.

Re: Location performance

2019-07-02 Thread Ralph Goers
But what do you think about includeLocation. Should it override the result of requiresLocation or should the fact that we determined location information is required override the includeLocation setting? Ralph > On Jul 2, 2019, at 2:34 PM, Matt Sicker wrote: > > These benchmarks are impressiv

Re: Location performance

2019-07-02 Thread Matt Sicker
These benchmarks are impressive! Moving the point of call site calculation is a great idea. On Tue, 2 Jul 2019 at 16:22, Ralph Goers wrote: > I have made changes to the code the master branch to measure and improve > performance. I have committed some of the changes but not the last set. I > hav

Location performance

2019-07-02 Thread Ralph Goers
I have made changes to the code the master branch to measure and improve performance. I have committed some of the changes but not the last set. I have not made the latest changes to release-2.x because they will have to be implemented a bit differently to avoid compatibility issues. The table b