> On Aug. 14, 2014, 8:01 nachm., Sergey Edunov wrote: > > giraph-examples/src/main/java/org/apache/giraph/examples/scc/SccComputation.java, > > line 204 > > <https://reviews.apache.org/r/24665/diff/1/?file=659562#file659562line204> > > > > No need to create LongWritable here as well
Done! - Gianluca ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24665/#review50621 ----------------------------------------------------------- On Aug. 15, 2014, 8:31 vorm., Gianluca Righetto wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24665/ > ----------------------------------------------------------- > > (Updated Aug. 15, 2014, 8:31 vorm.) > > > Review request for giraph. > > > Repository: giraph-git > > > Description > ------- > > Review the second version of the patch for the issue GIRAPH-931 (Provide a > Strongly Connected Components algorithm). > > > Diffs > ----- > > giraph-examples/pom.xml 90f6889 > > giraph-examples/src/main/java/org/apache/giraph/examples/scc/SccComputation.java > PRE-CREATION > > giraph-examples/src/main/java/org/apache/giraph/examples/scc/SccLongLongNullTextInputFormat.java > PRE-CREATION > > giraph-examples/src/main/java/org/apache/giraph/examples/scc/SccPhaseMasterCompute.java > PRE-CREATION > > giraph-examples/src/main/java/org/apache/giraph/examples/scc/SccVertexValue.java > PRE-CREATION > > giraph-examples/src/main/java/org/apache/giraph/examples/scc/package-info.java > PRE-CREATION > > giraph-examples/src/test/java/org/apache/giraph/examples/scc/SccComputationTestInMemory.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/24665/diff/ > > > Testing > ------- > > This has been tested on a real cluster using two of the Stanford's SNAP > datasets. Also, a new in-memory test is provided in the patch. > > > Thanks, > > Gianluca Righetto > >