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

move traceMask calculation logic into the trace log guard

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • 3.8.0
    • 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

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

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

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