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

Avery Ching commented on GIRAPH-722:
------------------------------------

+1.

Regarding your comment.  Couldn't the same thing happen for waitFor() where it 
times out before calling progress?  I guess semantically, should waitFor() 
guarantee that progress() is called, no matter the outcome.  Anyway, this is 
great.  Thanks for the test too.
                
> ProgressableUtils.waitForever is not calling progress
> -----------------------------------------------------
>
>                 Key: GIRAPH-722
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-722
>             Project: Giraph
>          Issue Type: Bug
>            Reporter: Maja Kabiljo
>            Assignee: Maja Kabiljo
>         Attachments: GIRAPH-722.patch, GIRAPH-722.patch
>
>
> ProgressableUtils.waitForever is not calling progress at all, because waitFor 
> which it calls periodically times out until result is ready.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to