Re: Review Request: Multigraph support.

2012-12-18 Thread Nitay Joffe
> On Dec. 19, 2012, 12:26 a.m., Nitay Joffe wrote: > > /trunk/giraph/src/main/java/org/apache/giraph/graph/EdgeListVertexBase.java, > > line 55 > > > > > > Why is this not just a default implementation of addEdge()? >

Re: Review Request: Multigraph support.

2012-12-18 Thread Alessandro Presta
> On Dec. 19, 2012, 12:26 a.m., Nitay Joffe wrote: > > /trunk/giraph/src/main/java/org/apache/giraph/graph/EdgeListVertexBase.java, > > line 55 > > > > > > Why is this not just a default implementation of addEdge()? I

Re: Review Request: Multigraph support.

2012-12-18 Thread Nitay Joffe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8560/#review14698 --- This code looks great. I have a few minor points but the overall big

Re: Review Request: Multigraph support.

2012-12-12 Thread Alessandro Presta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8560/ --- (Updated Dec. 13, 2012, 2:57 a.m.) Review request for giraph. Changes ---