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

Eli Reisman commented on GIRAPH-323:
------------------------------------

+1, committed. Thanks Maja!

                
> Check if requests are done before calling wait
> ----------------------------------------------
>
>                 Key: GIRAPH-323
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-323
>             Project: Giraph
>          Issue Type: Bug
>            Reporter: Maja Kabiljo
>            Assignee: Maja Kabiljo
>            Priority: Minor
>         Attachments: GIRAPH-323.patch
>
>
> In NettyClient.waitSomeRequests we don't check whether we still have open requests before calling wait. So we can end up waiting for another waitingRequestMsecs even though we are done, because last notifyAll was called before we started waiting.

--
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