[ 
https://issues.apache.org/jira/browse/GIRAPH-1141?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15951657#comment-15951657
 ] 

ASF GitHub Bot commented on GIRAPH-1141:
----------------------------------------

Github user majakabiljo commented on a diff in the pull request:

    https://github.com/apache/giraph/pull/33#discussion_r109251665
  
    --- Diff: 
giraph-core/src/main/java/org/apache/giraph/job/CombinedWorkerProgress.java ---
    @@ -204,4 +215,19 @@ public String toString() {
         }
         return sb.toString();
       }
    +
    +  /**
    +   * Check if this instance made progress from another instance
    +   *
    +   * @param lastProgress Instance to compare with
    +   * @return True iff progress was made
    +   */
    +  public boolean madeProgressFrom(CombinedWorkerProgress lastProgress) {
    --- End diff --
    
    That's why I separated getProgressString from toString, to only contain 
actual progress. For different supersteps we are looking at different numbers 
so this seemed the easiest to compare instead of having all the if-s.


> Kill the job if no progress is being made
> -----------------------------------------
>
>                 Key: GIRAPH-1141
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-1141
>             Project: Giraph
>          Issue Type: New Feature
>            Reporter: Maja Kabiljo
>            Assignee: Maja Kabiljo
>            Priority: Minor
>
> Sometimes jobs can get stuck for various reasons, it's better to have an 
> option to kill them then to keep them running holding resources.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to