[GitHub] activemq-artemis pull request #1668: ARTEMIS-1522 add date to timestamp in l...

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] activemq-artemis pull request #1668: ARTEMIS-1522 add date to timestamp in l...

RaiSaurabh
GitHub user pgfox opened a pull request:

    https://github.com/apache/activemq-artemis/pull/1668

    ARTEMIS-1522 add date to timestamp in logs

    Currently the date can derived from the log file prefix but in scenarios where you are only seeing snippets of log files or these log files are being manipulated later; it makes it much easier if the date is printed with the log entry.
   
    I understand people want to reduce log space but adding the date would avoid any potential ambiguity.
   
    current format
    ``
    11:56:49,051 INFO  [org.apache.activemq.artemis.core.server] AMQ221007: Server is now live
    ``
    Suggested new format
    ``
    2017-11-24 15:24:11,803 INFO  [org.apache.activemq.artemis.core.server] AMQ221007: Server is now live
    ``

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/pgfox/activemq-artemis date_format

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/activemq-artemis/pull/1668.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1668
   
----
commit ee035afa07c4544a45ff840321a500dc5153002a
Author: Pat Fox <[hidden email]>
Date:   2017-11-24T14:08:40Z

    ARTEMIS-1522 add date to timestamp in logs

----


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] activemq-artemis pull request #1668: ARTEMIS-1522 add date to timestamp in l...

RaiSaurabh
Github user asfgit closed the pull request at:

    https://github.com/apache/activemq-artemis/pull/1668


---