-
Notifications
You must be signed in to change notification settings - Fork 127
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#1428 Add timestamps for logs #1439
Conversation
feat(objectionary#1428): add simple log4j configuration feat(objectionary#1428): add test for log message format feat(objectionary#1428): adjust test format with desired format feat(objectionary#1428): implement the correct logging test feat(objectionary#1428): fix all qulice suggestions feat(objectionary#1428): use log4j.properties instead log4j.xml feat(objectionary#1428): fix dependencies
98cb5a4
to
9d3a7b0
Compare
045a283
to
7ff1ec8
Compare
feat(objectionary#1428): simplify log4.properties files feat(objectionary#1428): declare dependencies in a more straigtforward way feat(objectionary#1428): use OS independent line separator
7ff1ec8
to
81a4d80
Compare
@mximp pls, take a look |
@yegor256 merge please |
@@ -2,10 +2,10 @@ log4j.rootLogger=WARN, CONSOLE | |||
|
|||
log4j.appender.CONSOLE=org.apache.log4j.ConsoleAppender | |||
log4j.appender.CONSOLE.layout=com.jcabi.log.MulticolorLayout | |||
log4j.appender.CONSOLE.layout.ConversionPattern=[%color{%p}] %c: %m%n | |||
log4j.appender.CONSOLE.layout.ConversionPattern=%d{yyyy-MM-dd HH:mm:ss} [%color{%p}] %c: %m%n |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@volodya-lombrozo do we really need a year in tests? :) I think mm:ss
is more than enough.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yegor256 fixed. I've left only HH:mm:ss
. If we leave only mm:ss
- it will confuse, because looks like HH:mm
@@ -2,7 +2,7 @@ log4j.rootLogger=WARN, CONSOLE | |||
|
|||
log4j.appender.CONSOLE=org.apache.log4j.ConsoleAppender | |||
log4j.appender.CONSOLE.layout=com.jcabi.log.MulticolorLayout | |||
log4j.appender.CONSOLE.layout.ConversionPattern=[%color{%p}] %c: %m%n | |||
log4j.appender.CONSOLE.layout.ConversionPattern=%d{yyyy-MM-dd HH:mm:ss} [%p] %c: %m%n |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@volodya-lombrozo same here, why year/month/day?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yegor256 fixed. I've left only HH:mm:ss
. If we leave only mm:ss
- it will confuse, because looks like HH:mm
log4j.appender.CONSOLE.layout=org.apache.log4j.PatternLayout | ||
log4j.appender.CONSOLE.layout.ConversionPattern=BUILD %c: %m%n | ||
log4j.appender.CONSOLE.layout=com.jcabi.log.MulticolorLayout | ||
log4j.appender.CONSOLE.layout.ConversionPattern=%d{yyyy-MM-dd HH:mm:ss} [%color{%p}] %c: %m%n |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@volodya-lombrozo same here, why year/month/day?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yegor256 fixed. I've left only HH:mm:ss
. If we leave only mm:ss
- it will confuse, because looks like HH:mm
@rultor merge |
Added timestamps for
log4j
logger configuration and provided tests in order to prove that the new format is actually working.Closes: #1428
That PR adds timestamps only for
log4j
logs, that is useful for testing purposes. In order to add timestamps for maven build process we have to implement the next issue: objectionary/eoc#94