[
https://issues.apache.org/jira/browse/ZOOKEEPER-3942?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Chen Zhi updated ZOOKEEPER-3942:
--------------------------------
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. (was:
Similar to ZOOKEEPER-3708 and ZOOKEEPER-3728 in Learner, I found two cases
where `traceMask` calculation logic are out of trace log guard in
LearnerHandler. 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.)
> move traceMask calculation logic into the trace log guard
> ---------------------------------------------------------
>
> Key: ZOOKEEPER-3942
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3942
> Project: ZooKeeper
> Issue Type: Improvement
> Reporter: Chen Zhi
> Priority: Minor
>
> 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.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
