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

2017-11-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---


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

2017-11-24 Thread pgfox
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 
Date:   2017-11-24T14:08:40Z

ARTEMIS-1522 add date to timestamp in logs




---