This is an automated email from the ASF dual-hosted git repository. baunsgaard pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/systemds.git
commit 3f4868d1679ce10baf929f16207758c84a19881e Author: Sebastian Baunsgaard <[email protected]> AuthorDate: Mon Feb 16 15:47:24 2026 +0000 [MINOR] Reduce logging in MLContextTests Closes #2432 --- .../org/apache/sysds/test/applications/nn/BaseTest.java | 6 ------ .../sysds/test/functions/mlcontext/MLContextTestBase.java | 13 ------------- 2 files changed, 19 deletions(-) diff --git a/src/test/java/org/apache/sysds/test/applications/nn/BaseTest.java b/src/test/java/org/apache/sysds/test/applications/nn/BaseTest.java index f8b33fdc3e..b5312612b6 100644 --- a/src/test/java/org/apache/sysds/test/applications/nn/BaseTest.java +++ b/src/test/java/org/apache/sysds/test/applications/nn/BaseTest.java @@ -32,12 +32,6 @@ public abstract class BaseTest extends MLContextTestBase { private static final String ERROR_STRING = "ERROR:"; - public BaseTest() { - //disable debug and trace logging in mlcontext super class since - //the nn tests execute a lot of mini-batch operations - _enableTracing = false; - } - protected void run(String name) { run(name, false); } diff --git a/src/test/java/org/apache/sysds/test/functions/mlcontext/MLContextTestBase.java b/src/test/java/org/apache/sysds/test/functions/mlcontext/MLContextTestBase.java index b21bcaf54c..0a54ac1aab 100644 --- a/src/test/java/org/apache/sysds/test/functions/mlcontext/MLContextTestBase.java +++ b/src/test/java/org/apache/sysds/test/functions/mlcontext/MLContextTestBase.java @@ -26,8 +26,6 @@ import java.io.PrintStream; import org.apache.commons.lang3.tuple.ImmutablePair; import org.apache.commons.lang3.tuple.Pair; -import org.apache.log4j.Level; -import org.apache.log4j.Logger; import org.apache.spark.api.java.JavaSparkContext; import org.apache.spark.sql.SparkSession; import org.apache.sysds.api.mlcontext.MLContext; @@ -65,8 +63,6 @@ public abstract class MLContextTestBase extends AutomatedTestBase { protected String testDir = null; protected String testName = null; - protected Level _oldLevel = null; - protected boolean _enableTracing = true; @Override public void setUp() { @@ -76,13 +72,6 @@ public abstract class MLContextTestBase extends AutomatedTestBase { addTestConfiguration(dir, name); getAndLoadTestConfiguration(name); - - //run all mlcontext tests in loglevel trace to improve test coverage - //of all logging in various components - if( _enableTracing ) { - _oldLevel = Logger.getLogger("org.apache.sysds").getLevel(); - Logger.getLogger("org.apache.sysds").setLevel( Level.TRACE ); - } } @BeforeClass @@ -96,8 +85,6 @@ public abstract class MLContextTestBase extends AutomatedTestBase { @Override public void tearDown() { super.tearDown(); - if(_enableTracing) - Logger.getLogger("org.apache.sysds").setLevel( _oldLevel ); } @AfterClass
