You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by uc...@apache.org on 2015/10/06 11:43:01 UTC

flink git commit: [FLINK-2741] Use single log statement in TestLogger

Repository: flink
Updated Branches:
  refs/heads/master de14a8015 -> 2b81b17d5


[FLINK-2741] Use single log statement in TestLogger

This closes #1221


Project: http://git-wip-us.apache.org/repos/asf/flink/repo
Commit: http://git-wip-us.apache.org/repos/asf/flink/commit/2b81b17d
Tree: http://git-wip-us.apache.org/repos/asf/flink/tree/2b81b17d
Diff: http://git-wip-us.apache.org/repos/asf/flink/diff/2b81b17d

Branch: refs/heads/master
Commit: 2b81b17d5fea967e5b7be8fc00594e6144fdef89
Parents: de14a80
Author: Rerngvit Yanggratoke <re...@kth.se>
Authored: Mon Oct 5 13:35:42 2015 +0200
Committer: Ufuk Celebi <uc...@apache.org>
Committed: Tue Oct 6 11:42:56 2015 +0200

----------------------------------------------------------------------
 .../java/org/apache/flink/util/TestLogger.java  | 21 +++++++++++---------
 1 file changed, 12 insertions(+), 9 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/flink/blob/2b81b17d/flink-core/src/test/java/org/apache/flink/util/TestLogger.java
----------------------------------------------------------------------
diff --git a/flink-core/src/test/java/org/apache/flink/util/TestLogger.java b/flink-core/src/test/java/org/apache/flink/util/TestLogger.java
index 49d0f0f..7166d06 100644
--- a/flink-core/src/test/java/org/apache/flink/util/TestLogger.java
+++ b/flink-core/src/test/java/org/apache/flink/util/TestLogger.java
@@ -37,23 +37,26 @@ public class TestLogger {
 
 		@Override
 		public void starting(Description description) {
-			log.info("================================================================================");
-			log.info("Test {} is running.", description);
-			log.info("--------------------------------------------------------------------------------");
+			log.info("\n================================================================================"
+					+ "\nTest {} is running."
+					+ "\n--------------------------------------------------------------------------------"
+					, description);
 		}
 
 		@Override
 		public void succeeded(Description description) {
-			log.info("--------------------------------------------------------------------------------");
-			log.info("Test {} successfully run.", description);
-			log.info("================================================================================");
+			log.info("\n--------------------------------------------------------------------------------"
+					+ "\nTest {} successfully run."
+					+ "\n================================================================================"
+					, description);
 		}
 
 		@Override
 		public void failed(Throwable e, Description description) {
-			log.error("--------------------------------------------------------------------------------");
-			log.error("Test {} failed with:\n{}", description, StringUtils.stringifyException(e));
-			log.error("================================================================================");
+			log.error("\n--------------------------------------------------------------------------------"
+					 + "\nTest {} failed with:\n{}"
+					 + "\n================================================================================"
+					, description, StringUtils.stringifyException(e));
 		}
 	};
 }