Karaf
  1. Karaf
  2. KARAF-784

Add date in the default Karaf log

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.2.2
    • Fix Version/s: 2.2.3, 3.0.0
    • Component/s: karaf-core
    • Labels:
      None

      Description

      The default log pattern used in Karaf for the out file (karaf.log) is:

      %d

      {ABSOLUTE}

      | %-5.5p | %-16.16t | %-32.32c

      {1} | %-32.32C %4L | %X{bundle.id} - %X{bundle.name} - %X{bundle.version} | %m%n

      It means that a log line looks like:

      16:36:51,285 | INFO | Thread-6 | FeaturesServiceImpl

      As Karaf is a container, and is up for a "long" time, it makes sense to add the date in the log. The log line will looks like:

      2011-07-27 16:49:16,557 | INFO | Thread-4 | FeaturesServiceImpl |

      I propose to use the following pattern (at least in the karaf.log file):

      %d{yyyy-MM-dd HH:mm:ss,SSS} | %-5.5p | %-16.16t | %-32.32c{1}

      | %-32.32C %4L | %X

      {bundle.id}

      - %X

      {bundle.name}

      - %X

      {bundle.version}

      | %m%n

        Activity

        Hide
        Jean-Baptiste Onofré added a comment -

        Fixed on karaf-2.2.x branch: revision 1152239.

        Show
        Jean-Baptiste Onofré added a comment - Fixed on karaf-2.2.x branch: revision 1152239.
        Hide
        Jean-Baptiste Onofré added a comment -

        Fixed on trunk: revision 1152230.

        Show
        Jean-Baptiste Onofré added a comment - Fixed on trunk: revision 1152230.
        Hide
        Jean-Baptiste Onofré added a comment -

        Yes, I put %d

        {yyyy-MM-dd HH:mm:ss,SSS}

        just for information about what is the ISO8601 format.

        Thanks for the precision Glen.

        Show
        Jean-Baptiste Onofré added a comment - Yes, I put %d {yyyy-MM-dd HH:mm:ss,SSS} just for information about what is the ISO8601 format. Thanks for the precision Glen.
        Hide
        Glen Mazza added a comment -

        I assume you mean %d

        {ISO8601}, correct? From here: http://logging.apache.org/log4j/1.2/apidocs/org/apache/log4j/PatternLayout.html "%d{ISO8601}

        " apparently activates a faster parser than SimpleDateFormat.

        Show
        Glen Mazza added a comment - I assume you mean %d {ISO8601}, correct? From here: http://logging.apache.org/log4j/1.2/apidocs/org/apache/log4j/PatternLayout.html "%d{ISO8601} " apparently activates a faster parser than SimpleDateFormat.

          People

          • Assignee:
            Jean-Baptiste Onofré
            Reporter:
            Jean-Baptiste Onofré
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development