You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by gg...@apache.org on 2015/07/19 05:14:28 UTC

logging-log4j2 git commit: Adjustment after discussion with Remko on a previous patch causing uninteresting timings to be logged.

Repository: logging-log4j2
Updated Branches:
  refs/heads/master 7f075a710 -> 753b79df2


Adjustment after discussion with Remko on a previous patch causing
uninteresting timings to be logged.

Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/753b79df
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/753b79df
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/753b79df

Branch: refs/heads/master
Commit: 753b79df2a808100964e90dde32fdfee2ff93465
Parents: 7f075a7
Author: ggregory <gg...@apache.org>
Authored: Sat Jul 18 20:14:24 2015 -0700
Committer: ggregory <gg...@apache.org>
Committed: Sat Jul 18 20:14:24 2015 -0700

----------------------------------------------------------------------
 .../log4j/core/appender/rolling/DefaultRolloverStrategy.java     | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/753b79df/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/DefaultRolloverStrategy.java
----------------------------------------------------------------------
diff --git a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/DefaultRolloverStrategy.java b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/DefaultRolloverStrategy.java
index 10b2340..6da1b77 100644
--- a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/DefaultRolloverStrategy.java
+++ b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/DefaultRolloverStrategy.java
@@ -482,8 +482,8 @@ public class DefaultRolloverStrategy implements RolloverStrategy {
             return null;
         }
         if (LOGGER.isTraceEnabled()) {
-            final double durationSeconds = TimeUnit.NANOSECONDS.toSeconds(System.nanoTime() - startNanos);
-            LOGGER.trace("DefaultRolloverStrategy.purge() took {} seconds", durationSeconds);
+            final double durationMillis = TimeUnit.NANOSECONDS.toMillis(System.nanoTime() - startNanos);
+            LOGGER.trace("DefaultRolloverStrategy.purge() took {} milliseconds", durationMillis);
         }
         final StringBuilder buf = new StringBuilder(255);
         manager.getPatternProcessor().formatFileName(subst, buf, fileIndex);