osiegmar/logback-gelf

Faster MessageIdSupplier with less object allocation, error-prone

magicprinc opened this issue · 1 comments

magicprinc commented

  1. MessageIdSupplier Supplier → LongSupplier: less object allocation (temporary Longs)

  2. System.nanoTime is quite slow → AtomicLong with random starting value

  3. error-prone (google static code analyzer / linter)

  4. error-prone has found some minor problems: fixed

https://github.com/magicprinc/logback-gelf/commits/feature/messageIdSupplier

I would send it as PR later

osiegmar commented

Thanks for the hint on LongSuppliert – implemented it by f426aeb. Will react on the PR later.

  • 👍1