bookkeeper icon indicating copy to clipboard operation
bookkeeper copied to clipboard

[improve] Optimize log4j configuration

Open Shawyeok opened this issue 3 years ago • 3 comments

Motivation

Log output with location information is very expensive, we should avoid use it in our log configuration. In most cases, we are able to find which line print specific log by logger name and log content.

Changes

  • remove log line information
  • use logger name instead of caller class name

Shawyeok avatar Oct 27 '22 17:10 Shawyeok

If this pr merged, I would like to port to other release branches.

Shawyeok avatar Oct 27 '22 17:10 Shawyeok

ping @Shawyeok, would you please take a look at the comments? thanks a lot.

hangc0276 avatar Jan 11 '23 07:01 hangc0276

@horizonzy Would you please help take a look at this PR? Thanks a lot.

hangc0276 avatar Jan 11 '23 07:01 hangc0276