[jira] [Created] (GIRAPH-1159) DefaultJobProgressTrackerService: Only kill the job if it's still running

2017-09-15 Thread Maja Kabiljo (JIRA)
Maja Kabiljo created GIRAPH-1159:


 Summary: DefaultJobProgressTrackerService: Only kill the job if 
it's still running
 Key: GIRAPH-1159
 URL: https://issues.apache.org/jira/browse/GIRAPH-1159
 Project: Giraph
  Issue Type: Bug
Reporter: Maja Kabiljo
Assignee: Maja Kabiljo


In killJobWithMessage, we need to check if the job has completed before killing 
it.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (GIRAPH-1159) DefaultJobProgressTrackerService: Only kill the job if it's still running

2017-09-15 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on GIRAPH-1159:


GitHub user majakabiljo opened a pull request:

https://github.com/apache/giraph/pull/48

GIRAPH-1159

DefaultJobProgressTrackerService: Only kill the job if it's still running

Summary: In killJobWithMessage, we need to check if the job has completed 
before killing it.

Test Plan: Verified kill is not called on already completed job

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/majakabiljo/giraph giraph-1159

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/giraph/pull/48.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #48


commit 30345e38bb6f53e8f629d1902e7d6e3de7730ce1
Author: Maja Kabiljo 
Date:   2017-09-15T17:05:16Z

GIRAPH-1159: DefaultJobProgressTrackerService: Only kill the job if it's 
still running

Summary: In killJobWithMessage, we need to check if the job has completed 
before killing it.

Test Plan: Verified kill is not called on already completed job




> DefaultJobProgressTrackerService: Only kill the job if it's still running
> -
>
> Key: GIRAPH-1159
> URL: https://issues.apache.org/jira/browse/GIRAPH-1159
> Project: Giraph
>  Issue Type: Bug
>Reporter: Maja Kabiljo
>Assignee: Maja Kabiljo
>
> In killJobWithMessage, we need to check if the job has completed before 
> killing it.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (GIRAPH-1158) Allow JobProgressTrackers to provide job completion estimations to fail early.

2017-09-15 Thread Hudson (JIRA)

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

Hudson commented on GIRAPH-1158:


FAILURE: Integrated in Jenkins build Giraph-trunk-Commit #1716 (See 
[https://builds.apache.org/job/Giraph-trunk-Commit/1716/])
GIRAPH-1158 (majakabiljo: 
[http://git-wip-us.apache.org/repos/asf?p=giraph.git&a=commit&h=d4db3a650fa40ee89513ff581653dbe2d44cb671])
* (edit) 
giraph-core/src/main/java/org/apache/giraph/job/DefaultJobProgressTrackerService.java
* (edit) 
giraph-core/src/main/java/org/apache/giraph/job/CombinedWorkerProgress.java


> Allow JobProgressTrackers to provide job completion estimations to fail early.
> --
>
> Key: GIRAPH-1158
> URL: https://issues.apache.org/jira/browse/GIRAPH-1158
> Project: Giraph
>  Issue Type: Task
>Reporter: Yuksel Akinci
>
> Changes in DefaultJobProgressTrackerService to enable derived classes to 
> provide job completion estimations and fail the job earlier if the job can't 
> be completed in allotted time. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)