[ZOOKEEPER-3942] move traceMask calculation logic into the trace log guard

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.8.0
• Labels: easyfix pull-request-available
• Assignee: Unassigned
• Reporter: Chen Zhi
• Created: 22/Sep/20 16:11
• Updated: 23/Jan/21 17:45
• Resolved: 23/Jan/21 17:45

Description

Similar to ZOOKEEPER-3708 and ZOOKEEPER-3728 in Learner,  I found four cases where traceMask calculation logic are out of trace log guard in LearnerHandler, PrepRequestProcessor, and ReadOnlyRequestProcessor. The traceMask only matters if trace is enabled, so move the associated code into the logging guard. The fix is simple and I will create a PR later.

Issue Links

• links to: GitHub Pull Request #1464
• links to: GitHub Pull Request #1464