[GEODE-5997] FastLogger optimization should work without using isEnabled clause

• Type: Wish
• Status: Closed
• Priority: Major
• Resolution: Won't Fix
• Component/s: logging
• Labels: pull-request-available
• Assignee: Kirk Lund
• Reporter: Kirk Lund
• Created: 06/Nov/18 18:16
• Updated: 10/Jul/19 23:48
• Resolved: 10/Jul/19 23:47

Description

Geode logger statements at debug or trace level currently have isDebugEnabled and isTraceEnabled clauses which are overridden by FastLogger to improve performance of disabled log statements.

If FastLogger is changed to override every method defined in the ExtendedLogger interface, then we can delete all isDebugEnabled and isTraceEnabled clauses.

It would be ideal to include new JMH benchmarks with any changes involving FastLogger.

Issue Links

• links to: GitHub Pull Request #2798