Avery Ching commented on GIRAPH-18:

Looks better, thanks.  Hey, could you add a javadoc comment for 
readVerticesFromInputSplit?  No need to attach the patch again, though, you can 
just check it in.

> Refactor BspServiceWorker::loadVertices()
> -----------------------------------------
>                 Key: GIRAPH-18
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-18
>             Project: Giraph
>          Issue Type: Improvement
>            Reporter: Jakob Homan
>            Assignee: Jakob Homan
>         Attachments: GIRAPH-18.patch, GIRAPH-18b.patch
> Currently BspServiceWorker::loadVertices() is more than 200 lines and 
> convoluted. I found it difficult to grok while debugging today.

This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira


Reply via email to