Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-3942

move traceMask calculation logic into the trace log guard

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None

      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.

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              coder_chenzhi Chen Zhi
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 0.5h
                0.5h