Uploaded image for project: 'Jetspeed (Retired)'
  1. Jetspeed (Retired)
  2. JS1-311

[ENH] Further logging improvements

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Resolution: Fixed
    • 1.4b5-dev / CVS
    • None
    • Miscellaneous
    • None
    • Operating System: All
      Platform: All
    • 19906

    Description

      This enhancement will consist of:

      Rewrite Mark's logging service into a logger factory and get rid of the use of
      the package org.apache.turbine.services.logging as it will cease to
      exist. Add a new wrappper class which will use Commons Logging, instead of
      using CL directly in all our code. It is then easier to change logging tool if
      wanted. I'll kee BaseLogger and led JetspeedLoggingService be backwards
      compatible until the changes are complete, so that pacthes for parts of the
      code base can be committed, and does not have to be done all at once.

      Change all use of org.apache.turbine.util.Log to our own log wrapper.

      Use a fine grained LOG4J configuration. Meaning something like this:
      static JetspeedLogger logger = JetspeedLoggingService.getLogger
      (<thisclass>.class);

      Provide LOG4J configuration file that shows Jetspeed users recommended ways to
      configure for debugging and for production use. LOG4J configuration will be
      supplied in both .properties and .xml versions, for users preferred choice.

      I marked this Bugzilla entry with severity "Major", as Mark wrote "...
      proposals are essential for the final release".

      I plan on finishing the work on this by the end of this month, and will most
      likely split the work between myself and James Nguyen, who has offered to lend
      a hand.

      Since so many committers have given their +1s, I'll go ahead and start right
      away.

      Harald

      Attachments

        Activity

          People

            Unassigned Unassigned
            haraldo Harald Ommang
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: