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

Eli Reisman commented on GIRAPH-228:
------------------------------------

ArrayListWritable patch, including change to POM to import edu.umd.cloud9 jar 
are done and pass mvn verify, but I am not uploading yet. If you guys sign off 
on adding the library, I can add it. Otherwise the existing patch that outputs 
text should be put into the repo to fix array probs with GIRAPH-217. Also 217 
might have similar problems now with the call to iterator() being removed from 
the API that my earier 228 patch did.

Thanks again for the advice, I think a multigraph example using 
ArrayListWritable for <E> param would be in order if this jar is ok to include 
in the project. Its also easier to work with in compute() than ArrayWritable. 
Yay!

                
> SimpleTriangleClosingVertex should not use ArrayWritable for a vertex value
> ---------------------------------------------------------------------------
>
>                 Key: GIRAPH-228
>                 URL: https://issues.apache.org/jira/browse/GIRAPH-228
>             Project: Giraph
>          Issue Type: Bug
>          Components: examples, test
>    Affects Versions: 0.2.0
>            Reporter: Eli Reisman
>            Assignee: Eli Reisman
>            Priority: Trivial
>             Fix For: 0.2.0
>
>         Attachments: GIRAPH-228-1.patch, GIRAPH-228-2.patch
>
>
> Need to convert to (yuck) Text output for vertex values, hope my tuples don't 
> get too long. I followed several instructions from Hadoop about subclassing 
> and using existing ArrayWritables, always both methods will compile with 'mvn 
> verify' nicely, but break in actual use. GiraphJob.run() seems to be the 
> culprit. Don't know if I can solve this myself in a way that will be pleasing 
> to everyone. See GIRAPH-227 if you can help fix this ArrayWritable problem. 
> As far as the vertex example, I'll covert it (and related tests) to Text 
> output now.
> Thanks again to Sean Choi, who noticed this and used the same (Text for 
> tuples) solution successfully. ;)
>  

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