You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by zentol <gi...@git.apache.org> on 2018/02/07 09:05:31 UTC

[GitHub] flink pull request #5419: [FLINK-8574][travis] Add timestamp to logging mess...

GitHub user zentol opened a pull request:

    https://github.com/apache/flink/pull/5419

    [FLINK-8574][travis] Add timestamp to logging messages

    With this PR logging statements on travis also include a timestamp( e.g. `09:00:27.972`). This allows us to better judge how long each part of build takes, in particular plugins.

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

    $ git pull https://github.com/zentol/flink 8574

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

    https://github.com/apache/flink/pull/5419.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 #5419
    
----
commit 4245b3069add4139562fadbd3c096f031f112dd2
Author: zentol <ch...@...>
Date:   2018-02-07T08:52:23Z

    [FLINK-8574][travis] Add timestamp to logging messages

----


---

[GitHub] flink issue #5419: [FLINK-8574][travis] Add timestamp to logging messages

Posted by greghogan <gi...@git.apache.org>.
Github user greghogan commented on the issue:

    https://github.com/apache/flink/pull/5419
  
    Can we also add timestamps to the surefire messages using `systemPropertyVariables`?
    
    https://gualtierotesta.wordpress.com/2015/11/01/tutorial-logging-during-tests/
    http://maven.apache.org/surefire/maven-surefire-plugin/examples/system-properties.html


---

[GitHub] flink issue #5419: [FLINK-8574][travis] Add timestamp to logging messages

Posted by zentol <gi...@git.apache.org>.
Github user zentol commented on the issue:

    https://github.com/apache/flink/pull/5419
  
    Looks like that isn't possible. As far as i can tell surefire does not use slfj4 but the plexus ConsoleLogger, and it looks like that one does not support timestamps.



---

[GitHub] flink pull request #5419: [FLINK-8574][travis] Add timestamp to logging mess...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/flink/pull/5419


---