You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@giraph.apache.org by di...@apache.org on 2020/09/02 16:13:23 UTC

[giraph] branch trunk updated: GIRAPH-1187

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

dionysios pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/giraph.git


The following commit(s) were added to refs/heads/trunk by this push:
     new b0d18a2  GIRAPH-1187
b0d18a2 is described below

commit b0d18a23a4445b51df767d53fda697b215d10776
Author: Randy Hu <ruweih>
AuthorDate: Wed Sep 2 09:12:47 2020 -0700

    GIRAPH-1187
    
    closes #142
---
 .../src/main/java/org/apache/giraph/graph/GraphTaskManager.java       | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/giraph-core/src/main/java/org/apache/giraph/graph/GraphTaskManager.java b/giraph-core/src/main/java/org/apache/giraph/graph/GraphTaskManager.java
index 2e35bcd..8bc73d3 100644
--- a/giraph-core/src/main/java/org/apache/giraph/graph/GraphTaskManager.java
+++ b/giraph-core/src/main/java/org/apache/giraph/graph/GraphTaskManager.java
@@ -642,7 +642,9 @@ end[PURE_YARN]*/
                   info.getGcCause() + ", duration = " +
                   info.getGcInfo().getDuration() + "ms");
             }
-            gcTimeMetric.inc(info.getGcInfo().getDuration());
+            if (gcTimeMetrics != null) {
+              gcTimeMetric.inc(info.getGcInfo().getDuration());
+            }
             GiraphMetrics.get().getGcTracker().gcOccurred(info.getGcInfo());
             for (GcObserver gcObserver : gcObservers) {
               gcObserver.gcOccurred(info);