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

Thomas Jungblut commented on HAMA-409:
--------------------------------------

I'm just trying to use it for inlinkcount.

Things I notice:

- Vertex class constructor is swallowing the exception (does the value be need 
to be in the basic vertex class?)
- Why are the setters avaiable? We should declare them package visible.
- SSSP and pagerank needs to check the args (minor thing)
- MessageIterator should have a hasNext() method.

We have to add the ability to write...

But otherwise good work. I like it.

                
> Add Pregel-like API
> -------------------
>
>                 Key: HAMA-409
>                 URL: https://issues.apache.org/jira/browse/HAMA-409
>             Project: Hama
>          Issue Type: New Feature
>          Components: bsp, examples
>    Affects Versions: 0.4.0
>            Reporter: Thomas Jungblut
>            Assignee: Edward J. Yoon
>              Labels: graph
>             Fix For: 0.5.0
>
>         Attachments: HAMA-409_v01.patch, HAMA-409_v02.patch, 
> HAMA-409_v03.patch, HAMA-409_v04.patch, HAMA-409_v05.patch, hama-graph.zip, 
> pregel_v01.patch, pregel_v02.patch, test.txt
>
>
> According to what we've discussed on the mailing list, we should add a 
> Pregel-like API.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to