This is an automated email from the ASF dual-hosted git repository.
Jackie-Jiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 7fb11d92fbf Fix LoggerUtilsTest flakiness caused by leaked
LoggerConfig (#19294)
7fb11d92fbf is described below
commit 7fb11d92fbfdaf1d2b6a89e8da6a3503e06ea1f2
Author: Xiaotian (Jackie) Jiang <[email protected]>
AuthorDate: Tue Aug 18 16:16:19 2026 -0700
Fix LoggerUtilsTest flakiness caused by leaked LoggerConfig (#19294)
---
.../org/apache/pinot/sql/parsers/SqlQueryOptionValidationTest.java | 6 ++++++
1 file changed, 6 insertions(+)
diff --git
a/pinot-common/src/test/java/org/apache/pinot/sql/parsers/SqlQueryOptionValidationTest.java
b/pinot-common/src/test/java/org/apache/pinot/sql/parsers/SqlQueryOptionValidationTest.java
index a69160a31e6..fe7cefde8c7 100644
---
a/pinot-common/src/test/java/org/apache/pinot/sql/parsers/SqlQueryOptionValidationTest.java
+++
b/pinot-common/src/test/java/org/apache/pinot/sql/parsers/SqlQueryOptionValidationTest.java
@@ -209,6 +209,12 @@ public class SqlQueryOptionValidationTest {
void detach() {
_logger.removeAppender(this);
stop();
+ // The captured logger is never declared in log4j2.xml, so attaching the
appender implicitly registered a
+ // LoggerConfig for it. Remove it to leave the shared configuration as
found — test classes share the JVM in
+ // this module, and other tests assert on the set of configured loggers.
+ LoggerContext context = _logger.getContext();
+ context.getConfiguration().removeLogger(_logger.getName());
+ context.updateLoggers();
}
List<String> messagesContaining(String substring) {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
