Uploaded image for project: 'JGroups'
  1. JGroups
  2. JGRP-120

Replace all occurrences of log.isTraceEnabled() with simple boolean

    XMLWordPrintable

Details

    • Task
    • Resolution: Done
    • Major
    • 2.2.9
    • 2.2.8
    • None

    Description

      Example:

      if(log.isTraceEnabled())

      should be replaced with

      if(trace)

      trace is initialized once at startup.
      Analyzing commons-logging and log4j, there is more work done in those calls than meets the eye, doing this in every down()
      or up() method adds significant overhead.

      Attachments

        Activity

          People

            rhn-engineering-bban Bela Ban
            rhn-engineering-bban Bela Ban
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: