You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@causeway.apache.org by da...@apache.org on 2024/06/19 12:21:58 UTC

(causeway) 07/09: CAUSEWAY-3781: simplifies default impl of LogWriter

This is an automated email from the ASF dual-hosted git repository.

danhaywood pushed a commit to branch maintenance-branch
in repository https://gitbox.apache.org/repos/asf/causeway.git

commit 90ff8447144458bf39723b8eb97a617876defc5a
Author: Dan Haywood <da...@haywood-associates.co.uk>
AuthorDate: Tue Jun 18 21:58:57 2024 +0100

    CAUSEWAY-3781: simplifies default impl of LogWriter
---
 .../metamodel/services/logwriter/LogWriter.java    | 24 +---------------------
 1 file changed, 1 insertion(+), 23 deletions(-)

diff --git a/core/metamodel/src/main/java/org/apache/causeway/core/metamodel/services/logwriter/LogWriter.java b/core/metamodel/src/main/java/org/apache/causeway/core/metamodel/services/logwriter/LogWriter.java
index 42843334c8..0b4073c8e9 100644
--- a/core/metamodel/src/main/java/org/apache/causeway/core/metamodel/services/logwriter/LogWriter.java
+++ b/core/metamodel/src/main/java/org/apache/causeway/core/metamodel/services/logwriter/LogWriter.java
@@ -56,33 +56,11 @@ public interface LogWriter {
 
         static final int MAX_ARG_LENGTH_AS_STR = 128;
 
-        private int repeating = 0;
-        private LogMessage previous = null;
-
-        @Value(staticConstructor = "of")
-        static class LogMessage {
-            String template;
-            Object[] args;
-        }
-
         @Override
         public void info(Logger log, String template, Object... args) {
             if(log.isInfoEnabled()) {
                 val argArray = Arrays.stream(args).map(this::asStrTruncateIfTooLong).toArray();
-                val logMessage = LogMessage.of(template, argArray);
-
-                if (logMessage.equals(previous)) {
-                    repeating++;
-                } else {
-                    if (repeating > 0) {
-                        log.info("... repeated {} time{}", repeating, (repeating > 1 ? "s" : ""));
-                    }
-
-                    repeating = 0;
-                    log.info(template, argArray);
-                }
-
-                previous = logMessage;
+                log.info(template, argArray);
             }
         }