You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@systemds.apache.org by ba...@apache.org on 2022/09/29 21:19:11 UTC

[systemds] 02/02: [MINOR] Add GitHub tests for monitoring tool

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 4cbcf88f67823e14a8d1d635718b015919f91475
Author: baunsgaard <ba...@tugraz.at>
AuthorDate: Thu Sep 29 23:13:43 2022 +0200

    [MINOR] Add GitHub tests for monitoring tool
    
    This commit adds GitHub actions for running federated tests,
    that previously were not added to the cloud execution of tests.
    Unfortunately some tests were already failing before the test were
    enabled in GitHub actions therefore the addition of these tests
    in actions on main branch is valid even if they are failing.
---
 .github/workflows/functionsTests.yml              | 3 ++-
 src/main/java/org/apache/sysds/api/DMLScript.java | 4 ++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/functionsTests.yml b/.github/workflows/functionsTests.yml
index 344b821bbc..d1967b22cf 100644
--- a/.github/workflows/functionsTests.yml
+++ b/.github/workflows/functionsTests.yml
@@ -55,7 +55,8 @@ jobs:
           "**.functions.binary.matrix_full_cellwise.**,**.functions.binary.matrix_full_other.**",
           "**.functions.federated.algorithms.**,**.functions.federated.io.**,**.functions.federated.paramserv.**",
           "**.functions.federated.primitives.**,**.functions.federated.transform.**",
-          "**.functions.federated.codegen.**",
+          "**.functions.federated.monitoring.**,**.functions.federated.multitenant",
+          "**.functions.federated.codegen.**,**.functions.federated.FederatedTestObjectConstructor",
           "**.functions.codegenalg.partone.**",
           "**.functions.builtin.part1.**",
           "**.functions.builtin.part2.**",
diff --git a/src/main/java/org/apache/sysds/api/DMLScript.java b/src/main/java/org/apache/sysds/api/DMLScript.java
index 671f2bbcde..e10335d8ba 100644
--- a/src/main/java/org/apache/sysds/api/DMLScript.java
+++ b/src/main/java/org/apache/sysds/api/DMLScript.java
@@ -36,7 +36,6 @@ import java.util.Date;
 import java.util.Map;
 import java.util.Scanner;
 
-import com.fasterxml.jackson.databind.ObjectMapper;
 import org.apache.commons.cli.AlreadySelectedException;
 import org.apache.commons.cli.HelpFormatter;
 import org.apache.commons.lang.StringUtils;
@@ -72,7 +71,6 @@ import org.apache.sysds.runtime.controlprogram.federated.FederatedData;
 import org.apache.sysds.runtime.controlprogram.federated.FederatedWorker;
 import org.apache.sysds.runtime.controlprogram.federated.monitoring.FederatedMonitoringServer;
 import org.apache.sysds.runtime.controlprogram.federated.monitoring.models.CoordinatorModel;
-import org.apache.sysds.runtime.controlprogram.federated.monitoring.models.WorkerModel;
 import org.apache.sysds.runtime.controlprogram.parfor.stat.InfrastructureAnalyzer;
 import org.apache.sysds.runtime.controlprogram.parfor.util.IDHandler;
 import org.apache.sysds.runtime.instructions.gpu.context.GPUContextPool;
@@ -90,6 +88,8 @@ import org.apache.sysds.utils.Explain.ExplainType;
 import org.apache.sysds.utils.NativeHelper;
 import org.apache.sysds.utils.Statistics;
 
+import com.fasterxml.jackson.databind.ObjectMapper;
+
 public class DMLScript 
 {
 	// Set the execution mode