[systemds] branch main updated: [MINOR] Fix corrupted -stats output (unchecked federated statistics)

[mboehm7]

This is an automated email from the ASF dual-hosted git repository.

mboehm7 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/systemds.git


The following commit(s) were added to refs/heads/main by this push:
     new 674b4e5941 [MINOR] Fix corrupted -stats output (unchecked federated 
statistics)
674b4e5941 is described below

commit 674b4e5941f81782e57bcfa05b64b6c89860d55f
Author: Matthias Boehm <mboe...@gmail.com>
AuthorDate: Fri Aug 5 16:10:07 2022 +0200

    [MINOR] Fix corrupted -stats output (unchecked federated statistics)
---
 .../runtime/controlprogram/federated/FederatedStatistics.java    | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git 
a/src/main/java/org/apache/sysds/runtime/controlprogram/federated/FederatedStatistics.java
 
b/src/main/java/org/apache/sysds/runtime/controlprogram/federated/FederatedStatistics.java
index b53ef801a5..9620648630 100644
--- 
a/src/main/java/org/apache/sysds/runtime/controlprogram/federated/FederatedStatistics.java
+++ 
b/src/main/java/org/apache/sysds/runtime/controlprogram/federated/FederatedStatistics.java
@@ -272,9 +272,12 @@ public class FederatedStatistics {
                sb.append(displayFedReuseReadStats());
                sb.append(displayFedPutLineageStats());
                sb.append(displayFedSerializationReuseStats());
-               sb.append(displayFedTransfer());
-               sb.append(displayCPUUsage());
-               sb.append(displayMemoryUsage());
+               //FIXME: the following statistics need guards to only show
+               // results if federated operations where executed, also the CPU
+               // and mem usage only probe once at the time of stats printing
+               //sb.append(displayFedTransfer());
+               //sb.append(displayCPUUsage());
+               //sb.append(displayMemoryUsage());
                return sb.toString();
        }