You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@systemds.apache.org by mb...@apache.org on 2022/08/05 14:10:26 UTC
[systemds] branch main updated: [MINOR] Fix corrupted -stats output (unchecked federated statistics)
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 <mb...@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();
}