giraph-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alessandro Presta" <alessan...@fb.com>
Subject Re: Review Request: Allow creation of graph by adding edges that span multiple workers
Date Fri, 02 Nov 2012 15:53:25 GMT


> On Nov. 2, 2012, 3:29 p.m., Nitay Joffe wrote:
> > /trunk/giraph/src/main/java/org/apache/giraph/graph/BspServiceWorker.java, line
70
> > <https://reviews.apache.org/r/7784/diff/2/?file=184009#file184009line70>
> >
> >     Is my IDE set to something different than yours formatting wise? Seems like
we should make the ordering a standard in our checkstyle or something and have it be the same
everywhere? Mine seems to alphabetically sort all third party imports together, followed by
all java.* imports. Yours seems to move non-org imports to separate section too?

We discussed it on GIRAPH-230, but there were arguments against it.
I can change my settings to match yours, but what about everybody else?
Feel free to revive that issue.


> On Nov. 2, 2012, 3:29 p.m., Nitay Joffe wrote:
> > /trunk/giraph/src/main/java/org/apache/giraph/graph/InputSplitEvents.java, line
36
> > <https://reviews.apache.org/r/7784/diff/2/?file=184019#file184019line36>
> >
> >     I think we can remove the inputSplit prefix since the class already says it.

Good idea.


- Alessandro


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7784/#review13038
-----------------------------------------------------------


On Nov. 2, 2012, 4:11 a.m., Alessandro Presta wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/7784/
> -----------------------------------------------------------
> 
> (Updated Nov. 2, 2012, 4:11 a.m.)
> 
> 
> Review request for giraph.
> 
> 
> Description
> -------
> 
> added EdgeInputFormat
> 
> 
> This addresses bug GIRAPH-155.
>     https://issues.apache.org/jira/browse/GIRAPH-155
> 
> 
> Diffs
> -----
> 
>   /trunk/giraph/src/main/java/org/apache/giraph/GiraphConfiguration.java 1404751 
>   /trunk/giraph/src/main/java/org/apache/giraph/ImmutableClassesGiraphConfiguration.java
1404751 
>   /trunk/giraph/src/main/java/org/apache/giraph/bsp/CentralizedServiceMaster.java 1404751

>   /trunk/giraph/src/main/java/org/apache/giraph/comm/WorkerServer.java 1404751 
>   /trunk/giraph/src/main/java/org/apache/giraph/comm/netty/NettyWorkerClientRequestProcessor.java
1404751 
>   /trunk/giraph/src/main/java/org/apache/giraph/comm/netty/NettyWorkerServer.java 1404751

>   /trunk/giraph/src/main/java/org/apache/giraph/comm/requests/SendPartitionMutationsRequest.java
1404751 
>   /trunk/giraph/src/main/java/org/apache/giraph/graph/BasicVertexValueReader.java PRE-CREATION

>   /trunk/giraph/src/main/java/org/apache/giraph/graph/BspService.java 1404751 
>   /trunk/giraph/src/main/java/org/apache/giraph/graph/BspServiceMaster.java 1404751 
>   /trunk/giraph/src/main/java/org/apache/giraph/graph/BspServiceWorker.java 1404751 
>   /trunk/giraph/src/main/java/org/apache/giraph/graph/BspUtils.java 1404751 
>   /trunk/giraph/src/main/java/org/apache/giraph/graph/EdgeInputFormat.java PRE-CREATION

>   /trunk/giraph/src/main/java/org/apache/giraph/graph/EdgeInputSplitsCallable.java PRE-CREATION

>   /trunk/giraph/src/main/java/org/apache/giraph/graph/EdgeInputSplitsCallableFactory.java
PRE-CREATION 
>   /trunk/giraph/src/main/java/org/apache/giraph/graph/EdgeReader.java PRE-CREATION 
>   /trunk/giraph/src/main/java/org/apache/giraph/graph/EdgeWithSource.java PRE-CREATION

>   /trunk/giraph/src/main/java/org/apache/giraph/graph/GiraphInputFormat.java PRE-CREATION

>   /trunk/giraph/src/main/java/org/apache/giraph/graph/GiraphJob.java 1404751 
>   /trunk/giraph/src/main/java/org/apache/giraph/graph/GraphMapper.java 1404751 
>   /trunk/giraph/src/main/java/org/apache/giraph/graph/InputSplitEvents.java PRE-CREATION

>   /trunk/giraph/src/main/java/org/apache/giraph/graph/InputSplitPaths.java PRE-CREATION

>   /trunk/giraph/src/main/java/org/apache/giraph/graph/InputSplitsCallable.java 1404751

>   /trunk/giraph/src/main/java/org/apache/giraph/graph/InputSplitsCallableFactory.java
PRE-CREATION 
>   /trunk/giraph/src/main/java/org/apache/giraph/graph/MasterThread.java 1404751 
>   /trunk/giraph/src/main/java/org/apache/giraph/graph/VertexInputFormat.java 1404751

>   /trunk/giraph/src/main/java/org/apache/giraph/graph/VertexInputSplitsCallable.java
PRE-CREATION 
>   /trunk/giraph/src/main/java/org/apache/giraph/graph/VertexInputSplitsCallableFactory.java
PRE-CREATION 
>   /trunk/giraph/src/main/java/org/apache/giraph/graph/VertexReader.java 1404751 
>   /trunk/giraph/src/main/java/org/apache/giraph/graph/VertexResolver.java 1404751 
>   /trunk/giraph/src/main/java/org/apache/giraph/graph/VertexValueInputFormat.java PRE-CREATION

>   /trunk/giraph/src/main/java/org/apache/giraph/graph/VertexValueReader.java PRE-CREATION

>   /trunk/giraph/src/main/java/org/apache/giraph/io/GiraphFileInputFormat.java PRE-CREATION

>   /trunk/giraph/src/main/java/org/apache/giraph/io/GiraphTextInputFormat.java PRE-CREATION

>   /trunk/giraph/src/main/java/org/apache/giraph/io/IntIntTextVertexValueInputFormat.java
PRE-CREATION 
>   /trunk/giraph/src/main/java/org/apache/giraph/io/IntNullTextEdgeInputFormat.java PRE-CREATION

>   /trunk/giraph/src/main/java/org/apache/giraph/io/TextEdgeInputFormat.java PRE-CREATION

>   /trunk/giraph/src/main/java/org/apache/giraph/io/TextVertexInputFormat.java 1404751

>   /trunk/giraph/src/main/java/org/apache/giraph/io/TextVertexValueInputFormat.java PRE-CREATION

>   /trunk/giraph/src/main/java/org/apache/giraph/utils/FileUtils.java 1404751 
>   /trunk/giraph/src/main/java/org/apache/giraph/utils/IntPair.java PRE-CREATION 
>   /trunk/giraph/src/main/java/org/apache/giraph/utils/InternalVertexRunner.java 1404751

>   /trunk/giraph/src/test/java/org/apache/giraph/BspCase.java 1404751 
>   /trunk/giraph/src/test/java/org/apache/giraph/TestEdgeInput.java PRE-CREATION 
>   /trunk/giraph/src/test/java/org/apache/giraph/TestJsonBase64Format.java 1404751 
> 
> Diff: https://reviews.apache.org/r/7784/diff/
> 
> 
> Testing
> -------
> 
> 1) mvn verify
> 2) pseudo-distributed tests
> 3) PageRank on cluster with real text input (will post perf results on the issue)
> 
> 
> Thanks,
> 
> Alessandro Presta
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message