[HBASE-25762] Improvement for some debug-logging guards

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.0.0-alpha-1, 2.5.0, 2.4.3, 2.3.6
• Component/s: logging, Performance
• Hadoop Flags: Reviewed
• Assignee: Chen Zhi
• Reporter: Chen Zhi
• Created: 11/Apr/21 02:24
• Updated: 14/Apr/21 18:31
• Resolved: 14/Apr/21 00:27

Description

Some potential expensive debug-logging calls are partially guarded by `isXXXEnabled()`. Therefore,

• Moving some potential expensive statements into the adjacent `isDebugEnabled` in `NettyRpcDuplexHandler.java`
• Moving the `isTraceEnabled` up to the beginning of `printChoreDetails` and `printChoreServiceDetails` in `ChoreService.java `.

Issue Links

• links to: GitHub Pull Request #3145