This is an automated email from the ASF dual-hosted git repository.
zhangliang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git
The following commit(s) were added to refs/heads/master by this push:
new ddc586a0940 Refactor LogCaptureExtension (#37034)
ddc586a0940 is described below
commit ddc586a094034fb7a86474d1b3829b5500969fce
Author: Liang Zhang <[email protected]>
AuthorDate: Fri Nov 7 13:14:55 2025 +0800
Refactor LogCaptureExtension (#37034)
* Refactor GroovyInlineExpressionParser
* Optimize GroovyInlineExpressionParser cache configuration and lock
mechanism
- Add GROOVY_INLINE_EXPRESSION_PARSING_CACHE_MAX_SIZE configuration property
- Replace hardcoded cache size with dynamic configuration support
- Implement double-checked locking pattern for updateMaxCacheSize method
- Add volatile variables to ensure thread safety
- Create comprehensive concurrency tests to verify thread safety
- Performance improvement: 2.39x faster for same cache size scenarios
- Maintain 100% backward compatibility with default cache size of 1000
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <[email protected]>
* Refactor GroovyInlineExpressionParser
* Update release notes
* Fix test cases
* Refactor LogCaptureExtension
---------
Co-authored-by: Claude <[email protected]>
---
.../metrics/type/prometheus/src/test/resources/logback-test.xml | 2 +-
infra/common/src/test/resources/logback-test.xml | 2 +-
infra/executor/src/test/resources/logback-test.xml | 4 ++--
kernel/transaction/type/xa/core/src/test/resources/logback-test.xml | 2 +-
.../type/xa/provider/atomikos/src/test/resources/logback-test.xml | 2 +-
test/e2e/driver/src/test/resources/logback-test.xml | 2 +-
test/e2e/fixture/src/test/assembly/conf/logback.xml | 2 +-
test/e2e/operation/pipeline/src/test/resources/env/logback.xml | 2 +-
.../resources/env/common/cluster/proxy/zookeeper/conf/logback.xml | 2 +-
.../src/test/resources/env/common/standalone/proxy/conf/logback.xml | 2 +-
.../test/infra/framework/extension/log/LogCaptureExtension.java | 4 ++--
test/it/rewriter/src/test/resources/logback-test.xml | 2 +-
12 files changed, 14 insertions(+), 14 deletions(-)
diff --git
a/agent/plugins/metrics/type/prometheus/src/test/resources/logback-test.xml
b/agent/plugins/metrics/type/prometheus/src/test/resources/logback-test.xml
index 2f7b3651007..0311c8d6167 100644
--- a/agent/plugins/metrics/type/prometheus/src/test/resources/logback-test.xml
+++ b/agent/plugins/metrics/type/prometheus/src/test/resources/logback-test.xml
@@ -27,4 +27,4 @@
<level value="warn" />
<appender-ref ref="console" />
</root>
-</configuration>
+</configuration>
diff --git a/infra/common/src/test/resources/logback-test.xml
b/infra/common/src/test/resources/logback-test.xml
index b420ff11210..88bbe7f0751 100644
--- a/infra/common/src/test/resources/logback-test.xml
+++ b/infra/common/src/test/resources/logback-test.xml
@@ -30,4 +30,4 @@
<level value="info" />
<appender-ref ref="console" />
</root>
-</configuration>
+</configuration>
diff --git a/infra/executor/src/test/resources/logback-test.xml
b/infra/executor/src/test/resources/logback-test.xml
index ae420403da1..b0686417570 100644
--- a/infra/executor/src/test/resources/logback-test.xml
+++ b/infra/executor/src/test/resources/logback-test.xml
@@ -34,7 +34,7 @@
</logger>
<root>
- <level value="error" />
+ <level value="INFO" />
<appender-ref ref="console" />
</root>
-</configuration>
+</configuration>
diff --git
a/kernel/transaction/type/xa/core/src/test/resources/logback-test.xml
b/kernel/transaction/type/xa/core/src/test/resources/logback-test.xml
index d17f7efbfaf..23cb4b8b539 100644
--- a/kernel/transaction/type/xa/core/src/test/resources/logback-test.xml
+++ b/kernel/transaction/type/xa/core/src/test/resources/logback-test.xml
@@ -31,4 +31,4 @@
<level value="error" />
<appender-ref ref="console" />
</root>
-</configuration>
+</configuration>
diff --git
a/kernel/transaction/type/xa/provider/atomikos/src/test/resources/logback-test.xml
b/kernel/transaction/type/xa/provider/atomikos/src/test/resources/logback-test.xml
index d17f7efbfaf..23cb4b8b539 100644
---
a/kernel/transaction/type/xa/provider/atomikos/src/test/resources/logback-test.xml
+++
b/kernel/transaction/type/xa/provider/atomikos/src/test/resources/logback-test.xml
@@ -31,4 +31,4 @@
<level value="error" />
<appender-ref ref="console" />
</root>
-</configuration>
+</configuration>
diff --git a/test/e2e/driver/src/test/resources/logback-test.xml
b/test/e2e/driver/src/test/resources/logback-test.xml
index a37d52c0e0f..649e4ee6147 100644
--- a/test/e2e/driver/src/test/resources/logback-test.xml
+++ b/test/e2e/driver/src/test/resources/logback-test.xml
@@ -32,4 +32,4 @@
<level value="error" />
<appender-ref ref="console" />
</root>
-</configuration>
+</configuration>
diff --git a/test/e2e/fixture/src/test/assembly/conf/logback.xml
b/test/e2e/fixture/src/test/assembly/conf/logback.xml
index e84a022d3ab..48116f137b0 100644
--- a/test/e2e/fixture/src/test/assembly/conf/logback.xml
+++ b/test/e2e/fixture/src/test/assembly/conf/logback.xml
@@ -36,4 +36,4 @@
<level value="error" />
<appender-ref ref="console" />
</root>
-</configuration>
+</configuration>
diff --git a/test/e2e/operation/pipeline/src/test/resources/env/logback.xml
b/test/e2e/operation/pipeline/src/test/resources/env/logback.xml
index bd0c4654000..3e5096022fb 100644
--- a/test/e2e/operation/pipeline/src/test/resources/env/logback.xml
+++ b/test/e2e/operation/pipeline/src/test/resources/env/logback.xml
@@ -37,4 +37,4 @@
<level value="ERROR" />
<appender-ref ref="console" />
</root>
-</configuration>
+</configuration>
diff --git
a/test/e2e/operation/showprocesslist/src/test/resources/env/common/cluster/proxy/zookeeper/conf/logback.xml
b/test/e2e/operation/showprocesslist/src/test/resources/env/common/cluster/proxy/zookeeper/conf/logback.xml
index 1494746120c..4f4c963e960 100644
---
a/test/e2e/operation/showprocesslist/src/test/resources/env/common/cluster/proxy/zookeeper/conf/logback.xml
+++
b/test/e2e/operation/showprocesslist/src/test/resources/env/common/cluster/proxy/zookeeper/conf/logback.xml
@@ -30,4 +30,4 @@
<level value="info" />
<appender-ref ref="console" />
</root>
-</configuration>
+</configuration>
diff --git
a/test/e2e/operation/showprocesslist/src/test/resources/env/common/standalone/proxy/conf/logback.xml
b/test/e2e/operation/showprocesslist/src/test/resources/env/common/standalone/proxy/conf/logback.xml
index 1494746120c..4f4c963e960 100644
---
a/test/e2e/operation/showprocesslist/src/test/resources/env/common/standalone/proxy/conf/logback.xml
+++
b/test/e2e/operation/showprocesslist/src/test/resources/env/common/standalone/proxy/conf/logback.xml
@@ -30,4 +30,4 @@
<level value="info" />
<appender-ref ref="console" />
</root>
-</configuration>
+</configuration>
diff --git
a/test/infra/framework/src/main/java/org/apache/shardingsphere/test/infra/framework/extension/log/LogCaptureExtension.java
b/test/infra/framework/src/main/java/org/apache/shardingsphere/test/infra/framework/extension/log/LogCaptureExtension.java
index d018e87af0e..50fa6ed3066 100644
---
a/test/infra/framework/src/main/java/org/apache/shardingsphere/test/infra/framework/extension/log/LogCaptureExtension.java
+++
b/test/infra/framework/src/main/java/org/apache/shardingsphere/test/infra/framework/extension/log/LogCaptureExtension.java
@@ -38,8 +38,8 @@ public final class LogCaptureExtension implements
BeforeEachCallback, AfterEachC
@Override
public void beforeEach(final ExtensionContext context) {
- String targetClassName =
context.getRequiredTestClass().getName().substring(0,
context.getRequiredTestClass().getName().lastIndexOf("Test"));
- logger = (Logger) LoggerFactory.getLogger(targetClassName);
+ String logTopic =
context.getRequiredTestClass().getName().substring(0,
context.getRequiredTestClass().getName().lastIndexOf("Test"));
+ logger = (Logger) LoggerFactory.getLogger(logTopic);
listAppender = new ListAppender<>();
listAppender.start();
logger.addAppender(listAppender);
diff --git a/test/it/rewriter/src/test/resources/logback-test.xml
b/test/it/rewriter/src/test/resources/logback-test.xml
index d17f7efbfaf..23cb4b8b539 100644
--- a/test/it/rewriter/src/test/resources/logback-test.xml
+++ b/test/it/rewriter/src/test/resources/logback-test.xml
@@ -31,4 +31,4 @@
<level value="error" />
<appender-ref ref="console" />
</root>
-</configuration>
+</configuration>
