revert custom LogLevels impl, not worth to add complexity, keep original simpler design

Signed-off-by: neo <1100909+neowu@users.noreply.github.com>

Author: neowu

CHANGELOG.md

@@ -2,7 +2,6 @@
 
 ### 9.3.1 (10/8/2025 - )
 
-* log: config output level via LogLevels.infoLevel()/traceLevel(), make it possible to reduce unnecessary external logs
 
 ### 9.3.0 (8/27/2025 - 10/8/2025) !!! only support java 25
 

core-ng/src/main/java/core/framework/internal/cache/LocalCacheStore.java

@@ -2,6 +2,7 @@
 
 import core.framework.internal.log.filter.ArrayLogParam;
 import core.framework.util.Maps;
+import org.jspecify.annotations.Nullable;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -18,12 +19,14 @@ public class LocalCacheStore implements CacheStore {
     private final Logger logger = LoggerFactory.getLogger(LocalCacheStore.class);
     public int maxSize = 10000;  // 10000 simple objects roughly takes 1M-10M heap + hashmap overhead
 
+    @Nullable
     @Override
     public <T> T get(String key, CacheContext<T> context) {
         logger.debug("get, key={}", key);
         return get(key, System.currentTimeMillis());
     }
 
+    @Nullable
     private <T> T get(String key, long now) {
         @SuppressWarnings("unchecked")
         CacheItem<T> item = (CacheItem<T>) caches.get(key);

core-ng/src/main/java/core/framework/internal/log/AbstractLogger.java

@@ -1,6 +1,5 @@
 package core.framework.internal.log;
 
-import core.framework.log.LogLevel;
 import org.slf4j.Logger;
 import org.slf4j.Marker;
 

core-ng/src/main/java/core/framework/internal/log/ActionLog.java

@@ -1,6 +1,5 @@
 package core.framework.internal.log;
 
-import core.framework.log.LogLevel;
 import core.framework.log.Markers;
 import core.framework.util.Strings;
 import org.jspecify.annotations.Nullable;
@@ -13,8 +12,8 @@
 import java.util.List;
 import java.util.Map;
 
-import static core.framework.log.LogLevel.DEBUG;
-import static core.framework.log.LogLevel.WARN;
+import static core.framework.internal.log.LogLevel.DEBUG;
+import static core.framework.internal.log.LogLevel.WARN;
 
 /**
  * @author neo

core-ng/src/main/java/core/framework/internal/log/DefaultLoggerFactory.java

@@ -1,7 +1,5 @@
 package core.framework.internal.log;
 
-import core.framework.log.LogLevel;
-import core.framework.log.LogLevels;
 import core.framework.util.Maps;
 import org.slf4j.ILoggerFactory;
 import org.slf4j.Logger;
@@ -13,13 +11,6 @@
  */
 public final class DefaultLoggerFactory implements ILoggerFactory {
     private final Map<String, Logger> loggers = Maps.newConcurrentHashMap();
-    private final LogLevels.Entry[] infoLevels;
-    private final LogLevels.Entry[] traceLevels;
-
-    public DefaultLoggerFactory(LogLevels.Entry[] infoLevels, LogLevels.Entry[] traceLevels) {
-        this.infoLevels = infoLevels;
-        this.traceLevels = traceLevels;
-    }
 
     @Override
     public Logger getLogger(String name) {
@@ -31,19 +22,22 @@ private Logger createLogger(String name) {
     }
 
     private LogLevel infoLevel(String name) {
-        for (LogLevels.Entry entry : infoLevels) {
-            if (name.startsWith(entry.prefix())) {
-                return entry.level();
-            }
+        // kafka log info for every producer/consumer, to reduce verbosity
+        if (name.startsWith("org.apache.kafka.")) {
+            return LogLevel.WARN;
+        }
+        // refer to org.elasticsearch.nativeaccess.NativeAccessHolder, to emmit warning under integration-test env
+        if (name.startsWith("org.elasticsearch.nativeaccess.")) {
+            return LogLevel.ERROR;
         }
         return LogLevel.INFO;
     }
 
     private LogLevel traceLevel(String name) {
-        for (LogLevels.Entry entry : traceLevels) {
-            if (name.startsWith(entry.prefix())) {
-                return entry.level();
-            }
+        if (name.startsWith("org.elasticsearch.")
+            || name.startsWith("org.mongodb.")
+            || name.startsWith("org.xnio.")) {
+            return LogLevel.INFO;
         }
         return LogLevel.DEBUG;
     }

core-ng/src/main/java/core/framework/internal/log/DefaultLoggerServiceProvider.java

@@ -1,6 +1,5 @@
 package core.framework.internal.log;
 
-import core.framework.log.LogLevels;
 import org.slf4j.ILoggerFactory;
 import org.slf4j.IMarkerFactory;
 import org.slf4j.helpers.BasicMarkerFactory;
@@ -38,7 +37,7 @@ public String getRequestedApiVersion() {
 
     @Override
     public void initialize() {
-        loggerFactory = LogLevels.createLoggerFactory();
+        loggerFactory = new DefaultLoggerFactory();
         markerFactory = new BasicMarkerFactory();
         mdcAdapter = new NOPMDCAdapter();
     }

core-ng/src/main/java/core/framework/internal/log/LogEvent.java

@@ -1,7 +1,6 @@
 package core.framework.internal.log;
 
 import core.framework.internal.log.marker.ErrorCodeMarker;
-import core.framework.log.LogLevel;
 import core.framework.util.Exceptions;
 import core.framework.util.Network;
 import org.jspecify.annotations.Nullable;

…in/java/core/framework/log/LogLevel.java …ore/framework/internal/log/LogLevel.javacore-ng/src/main/java/core/framework/log/LogLevel.java renamed to core-ng/src/main/java/core/framework/internal/log/LogLevel.java core-ng/src/main/java/core/framework/log/LogLevel.java renamed to core-ng/src/main/java/core/framework/internal/log/LogLevel.java

@@ -1,4 +1,4 @@
-package core.framework.log;
+package core.framework.internal.log;
 
 /**
  * @author neo

core-ng/src/main/java/core/framework/internal/log/LoggerImpl.java

@@ -1,6 +1,5 @@
 package core.framework.internal.log;
 
-import core.framework.log.LogLevel;
 import core.framework.util.Strings;
 import org.slf4j.Marker;