Qpid
  1. Qpid
  2. QPID-2840

Use a configurable message prefix on the AbstractActor class

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.7
    • Fix Version/s: 0.7
    • Component/s: Java Broker
    • Labels:
      None

      Description

      o.a.q.server.logging.actors.AbstractActor is using a DEFAULT_MSG_PREFIX of "MESSAGE" - when the broker is under intensive traffic we want to minimize the verbosity in the logging. This patch will make DEFAULT_MSG_PREFIX an empty string and will add a system property available for the user to modify it as needed.

        Activity

        Hide
        Robbie Gemmell added a comment -

        Patch applied, tests updated to use the property to ensure the expected prefix is available.

        Show
        Robbie Gemmell added a comment - Patch applied, tests updated to use the property to ensure the expected prefix is available.
        Hide
        Robbie Gemmell added a comment -

        The logging tests use the MESSAGE prefix (eg in AbstractTestLogging.getLog() ) for determining the actual log message by removing the prefixed log4j layout information. The test usage should be updated as necessary to ensure this functionality can continue to be used.

        Show
        Robbie Gemmell added a comment - The logging tests use the MESSAGE prefix (eg in AbstractTestLogging.getLog() ) for determining the actual log message by removing the prefixed log4j layout information. The test usage should be updated as necessary to ensure this functionality can continue to be used.

          People

          • Assignee:
            Robbie Gemmell
            Reporter:
            Sorin Suciu
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development