You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@giraph.apache.org by Nitay Joffe <ni...@apache.org> on 2013/06/06 21:20:36 UTC
Review Request: GIRAPH-468: Make Vertex an Interface (nitay)
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11682/
-----------------------------------------------------------
Review request for giraph.
Description
-------
Vertex interface
Diffs
-----
giraph-accumulo/src/test/java/org/apache/giraph/io/accumulo/edgemarker/AccumuloEdgeInputFormat.java 108ae61d5e57ba293a414028e1cb8b8c685b150c
giraph-core/src/main/java/org/apache/giraph/comm/SendMutationsCache.java 9348e61cd83985d920d3ed825d0fbdb3f3ab9fff
giraph-core/src/main/java/org/apache/giraph/comm/SendPartitionCache.java 68ce0955d8088ed87a6599fd5cd144cd7e470e6c
giraph-core/src/main/java/org/apache/giraph/comm/WorkerClientRequestProcessor.java 731d0eee0123ea9b5cecc47cff18d6be174f3e47
giraph-core/src/main/java/org/apache/giraph/comm/netty/NettyWorkerClientRequestProcessor.java 06965ca488aceb593a249a7c88bd49d660080c20
giraph-core/src/main/java/org/apache/giraph/comm/netty/NettyWorkerServer.java b45703853a6e5c57a5854cc40036912a20c59293
giraph-core/src/main/java/org/apache/giraph/conf/ImmutableClassesGiraphConfiguration.java aa5249875de551a341a4baac894c68553cbd6e63
giraph-core/src/main/java/org/apache/giraph/edge/EdgeStore.java 420bf9353871540594309ee9b4c129ae91ea7680
giraph-core/src/main/java/org/apache/giraph/graph/BasicVertex.java PRE-CREATION
giraph-core/src/main/java/org/apache/giraph/graph/Computation.java 84158df3ded55360af179e08194e14bc43949c77
giraph-core/src/main/java/org/apache/giraph/graph/ComputeCallable.java 6fdcfb05b102a78744d84bcba4e8dc8c30a3e25c
giraph-core/src/main/java/org/apache/giraph/graph/DefaultVertexResolver.java 748c3a6a2b84dc07d17bd453c1d3b680da24da26
giraph-core/src/main/java/org/apache/giraph/graph/IOVertex.java PRE-CREATION
giraph-core/src/main/java/org/apache/giraph/graph/Vertex.java 1241ae757b29cb7bec80fd3bb3d2bab1ac9c21bd
giraph-core/src/main/java/org/apache/giraph/graph/VertexChanges.java 3d09c06b43d2bd54d3c1ee70d2e4d6c9d3396207
giraph-core/src/main/java/org/apache/giraph/graph/VertexMutations.java 6f54dc701a5f5dc4589140dfb4a6d0dc5f128dca
giraph-core/src/main/java/org/apache/giraph/graph/VertexResolver.java b6659f47c6b6ce33efd0295487d34b0e2a0a3ffc
giraph-core/src/main/java/org/apache/giraph/io/VertexReader.java 9695169b5ccf8325987e42de90e7cd3011c0b901
giraph-core/src/main/java/org/apache/giraph/io/VertexValueReader.java 70e721ec8e4a120a094206e5b857b9bb75eb7acd
giraph-core/src/main/java/org/apache/giraph/io/formats/JsonLongDoubleFloatDoubleVertexInputFormat.java 112860c12d203b734093973c3b5d9dd6db39b14a
giraph-core/src/main/java/org/apache/giraph/io/formats/PseudoRandomIntNullVertexInputFormat.java d8abfdb063d745c736fe6c7559e8c709ed6fd115
giraph-core/src/main/java/org/apache/giraph/io/formats/PseudoRandomVertexInputFormat.java 91a19e6484631e4334b6c3843ba7aedfec169276
giraph-core/src/main/java/org/apache/giraph/io/formats/SequenceFileVertexInputFormat.java 1d31f4fb7a688ce314be5608f88726daa9152833
giraph-core/src/main/java/org/apache/giraph/io/formats/TextVertexInputFormat.java debdcccaf96cf105b5c0b3a73515f82860913178
giraph-core/src/main/java/org/apache/giraph/io/internal/WrappedVertexReader.java 54adfec054fb53abf9245fb458c9f456ad5f3e6f
giraph-core/src/main/java/org/apache/giraph/io/iterables/VertexReaderWrapper.java ca35c517666910e613431a3641f48f8df223f8b4
giraph-core/src/main/java/org/apache/giraph/partition/ByteArrayPartition.java 6eaa6d76efac80c53251ad1037f1c44ed6c4d933
giraph-core/src/main/java/org/apache/giraph/partition/DiskBackedPartitionStore.java dadce0f14fe4ecac1dd35c35adf593a63e9583d9
giraph-core/src/main/java/org/apache/giraph/partition/Partition.java aebd34344b10cb3752c74c27f62bf3cfa5d3d565
giraph-core/src/main/java/org/apache/giraph/partition/SimplePartition.java 0c1b40439a8891629b60aace4d89997ade42d1b5
giraph-core/src/main/java/org/apache/giraph/utils/InMemoryVertexInputFormat.java 6ca488cb42efec02b49f1ec9b4d22ec0b40bca42
giraph-core/src/main/java/org/apache/giraph/utils/TestGraph.java 6e46a76b656543c97a6b0c5eaad848633bd48bb6
giraph-core/src/main/java/org/apache/giraph/utils/WritableUtils.java c78d717fd1f1e91d2c07c6455ae671fba6516e82
giraph-core/src/main/java/org/apache/giraph/worker/VertexInputSplitsCallable.java 977e1000c33fd50c3074d535076772ad6ff1c746
giraph-core/src/test/java/org/apache/giraph/comm/RequestTest.java 70165729d5230a2620465ffbc69affc1896e1431
giraph-core/src/test/java/org/apache/giraph/graph/TestVertexAndEdges.java d0a6c46114b5d2b42a5fcdba93e293d404e4257f
giraph-core/src/test/java/org/apache/giraph/io/TestAdjacencyListTextVertexOutputFormat.java fdfb5e91dc692283a67f7fcaf309e8387f75a36d
giraph-core/src/test/java/org/apache/giraph/io/TestLongDoubleDoubleAdjacencyListVertexInputFormat.java c67a1c44eed7b35a3c872d2032bc4e5ae77e7ef7
giraph-core/src/test/java/org/apache/giraph/io/TestTextDoubleDoubleAdjacencyListVertexInputFormat.java 80340528326d9b37a380d9a761115b28b879ce2d
giraph-core/src/test/java/org/apache/giraph/partition/TestGiraphTransferRegulator.java 8a1e9edaea53476d50dd90d70afc80c76d7f090a
giraph-core/src/test/java/org/apache/giraph/partition/TestPartitionStores.java 45542b9083f8821ee664487791a7b6a50c115c8e
giraph-core/src/test/java/org/apache/giraph/utils/MockUtils.java d210928374b21e42bac960c2d191386df42c119e
giraph-examples/src/main/java/org/apache/giraph/examples/LongDoubleDoubleTextInputFormat.java f56b4f6a4c3f26e521829b5da4718ed8bc4e7b64
giraph-examples/src/main/java/org/apache/giraph/examples/LongDoubleNullTextInputFormat.java bfb5f40bd2e7b8da321b5b818e377fea16fccecd
giraph-examples/src/main/java/org/apache/giraph/examples/NormalizingLongDoubleDoubleTextInputFormat.java 5023a4e2ca80cf6fe0afe83598c11b00308fc630
giraph-examples/src/main/java/org/apache/giraph/examples/SimplePageRankComputation.java d053bb3ceb0c77a3a6dbe46ffa330e9d5f8d035c
giraph-examples/src/main/java/org/apache/giraph/examples/SimpleSuperstepComputation.java c3fd215b29a445b540a39b99f39cd7eaee195c91
giraph-examples/src/test/java/org/apache/giraph/TestBspBasic.java 28edbba58a0513a2fe0b051e02e723dfd283de20
giraph-examples/src/test/java/org/apache/giraph/examples/ConnectedComponentsComputationTestInMemory.java b94f8dcbd961d24f341cff252512c3a102b37ddb
giraph-examples/src/test/java/org/apache/giraph/examples/SimpleShortestPathsComputationTest.java 93a3248f7fd5a94a84c8daf7c277ca954c859071
giraph-examples/src/test/java/org/apache/giraph/examples/SimpleTriangleClosingComputationTest.java 73516a473ffd500500aeba083889ca3e5b7c3f03
giraph-hbase/src/test/java/org/apache/giraph/io/hbase/edgemarker/TableEdgeInputFormat.java 1af46d1d6f9e2add4eb25ecdf1e5b351b5bf9132
giraph-hbase/src/test/java/org/apache/giraph/io/hbase/edgemarker/TableEdgeOutputFormat.java a04d3864334158781b051e29fa8d02e3eba2b630
giraph-hcatalog/src/main/java/org/apache/giraph/io/hcatalog/HCatalogVertexInputFormat.java 53c7eed811e326980f8adf4a34ba41a4152b5416
giraph-hive/src/main/java/org/apache/giraph/hive/input/vertex/HiveToVertex.java a9736c69162e5d4a0a297cdea93b3e8186cdde5a
giraph-hive/src/main/java/org/apache/giraph/hive/input/vertex/HiveVertexReader.java db61afff4fffe03c90ed100da2edf5ee28c98c4e
giraph-hive/src/main/java/org/apache/giraph/hive/input/vertex/SimpleHiveToVertex.java 5483a6826d945a13a95dbd42eb8fa3e3aaeb2e97
Diff: https://reviews.apache.org/r/11682/diff/
Testing
-------
Thanks,
Nitay Joffe
Re: Review Request: GIRAPH-468: Make Vertex an Interface (nitay)
Posted by Nitay Joffe <ni...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11682/#review21566
-----------------------------------------------------------
giraph-core/src/main/java/org/apache/giraph/conf/ImmutableClassesGiraphConfiguration.java
<https://reviews.apache.org/r/11682/#comment44614>
Note I optimized this. There's no reason to use reflection and newInstance() when we know what the Vertex class will be.
- Nitay Joffe
On June 6, 2013, 7:20 p.m., Nitay Joffe wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/11682/
> -----------------------------------------------------------
>
> (Updated June 6, 2013, 7:20 p.m.)
>
>
> Review request for giraph.
>
>
> Description
> -------
>
> Vertex interface
>
>
> Diffs
> -----
>
> giraph-accumulo/src/test/java/org/apache/giraph/io/accumulo/edgemarker/AccumuloEdgeInputFormat.java 108ae61d5e57ba293a414028e1cb8b8c685b150c
> giraph-core/src/main/java/org/apache/giraph/comm/SendMutationsCache.java 9348e61cd83985d920d3ed825d0fbdb3f3ab9fff
> giraph-core/src/main/java/org/apache/giraph/comm/SendPartitionCache.java 68ce0955d8088ed87a6599fd5cd144cd7e470e6c
> giraph-core/src/main/java/org/apache/giraph/comm/WorkerClientRequestProcessor.java 731d0eee0123ea9b5cecc47cff18d6be174f3e47
> giraph-core/src/main/java/org/apache/giraph/comm/netty/NettyWorkerClientRequestProcessor.java 06965ca488aceb593a249a7c88bd49d660080c20
> giraph-core/src/main/java/org/apache/giraph/comm/netty/NettyWorkerServer.java b45703853a6e5c57a5854cc40036912a20c59293
> giraph-core/src/main/java/org/apache/giraph/conf/ImmutableClassesGiraphConfiguration.java aa5249875de551a341a4baac894c68553cbd6e63
> giraph-core/src/main/java/org/apache/giraph/edge/EdgeStore.java 420bf9353871540594309ee9b4c129ae91ea7680
> giraph-core/src/main/java/org/apache/giraph/graph/BasicVertex.java PRE-CREATION
> giraph-core/src/main/java/org/apache/giraph/graph/Computation.java 84158df3ded55360af179e08194e14bc43949c77
> giraph-core/src/main/java/org/apache/giraph/graph/ComputeCallable.java 6fdcfb05b102a78744d84bcba4e8dc8c30a3e25c
> giraph-core/src/main/java/org/apache/giraph/graph/DefaultVertexResolver.java 748c3a6a2b84dc07d17bd453c1d3b680da24da26
> giraph-core/src/main/java/org/apache/giraph/graph/IOVertex.java PRE-CREATION
> giraph-core/src/main/java/org/apache/giraph/graph/Vertex.java 1241ae757b29cb7bec80fd3bb3d2bab1ac9c21bd
> giraph-core/src/main/java/org/apache/giraph/graph/VertexChanges.java 3d09c06b43d2bd54d3c1ee70d2e4d6c9d3396207
> giraph-core/src/main/java/org/apache/giraph/graph/VertexMutations.java 6f54dc701a5f5dc4589140dfb4a6d0dc5f128dca
> giraph-core/src/main/java/org/apache/giraph/graph/VertexResolver.java b6659f47c6b6ce33efd0295487d34b0e2a0a3ffc
> giraph-core/src/main/java/org/apache/giraph/io/VertexReader.java 9695169b5ccf8325987e42de90e7cd3011c0b901
> giraph-core/src/main/java/org/apache/giraph/io/VertexValueReader.java 70e721ec8e4a120a094206e5b857b9bb75eb7acd
> giraph-core/src/main/java/org/apache/giraph/io/formats/JsonLongDoubleFloatDoubleVertexInputFormat.java 112860c12d203b734093973c3b5d9dd6db39b14a
> giraph-core/src/main/java/org/apache/giraph/io/formats/PseudoRandomIntNullVertexInputFormat.java d8abfdb063d745c736fe6c7559e8c709ed6fd115
> giraph-core/src/main/java/org/apache/giraph/io/formats/PseudoRandomVertexInputFormat.java 91a19e6484631e4334b6c3843ba7aedfec169276
> giraph-core/src/main/java/org/apache/giraph/io/formats/SequenceFileVertexInputFormat.java 1d31f4fb7a688ce314be5608f88726daa9152833
> giraph-core/src/main/java/org/apache/giraph/io/formats/TextVertexInputFormat.java debdcccaf96cf105b5c0b3a73515f82860913178
> giraph-core/src/main/java/org/apache/giraph/io/internal/WrappedVertexReader.java 54adfec054fb53abf9245fb458c9f456ad5f3e6f
> giraph-core/src/main/java/org/apache/giraph/io/iterables/VertexReaderWrapper.java ca35c517666910e613431a3641f48f8df223f8b4
> giraph-core/src/main/java/org/apache/giraph/partition/ByteArrayPartition.java 6eaa6d76efac80c53251ad1037f1c44ed6c4d933
> giraph-core/src/main/java/org/apache/giraph/partition/DiskBackedPartitionStore.java dadce0f14fe4ecac1dd35c35adf593a63e9583d9
> giraph-core/src/main/java/org/apache/giraph/partition/Partition.java aebd34344b10cb3752c74c27f62bf3cfa5d3d565
> giraph-core/src/main/java/org/apache/giraph/partition/SimplePartition.java 0c1b40439a8891629b60aace4d89997ade42d1b5
> giraph-core/src/main/java/org/apache/giraph/utils/InMemoryVertexInputFormat.java 6ca488cb42efec02b49f1ec9b4d22ec0b40bca42
> giraph-core/src/main/java/org/apache/giraph/utils/TestGraph.java 6e46a76b656543c97a6b0c5eaad848633bd48bb6
> giraph-core/src/main/java/org/apache/giraph/utils/WritableUtils.java c78d717fd1f1e91d2c07c6455ae671fba6516e82
> giraph-core/src/main/java/org/apache/giraph/worker/VertexInputSplitsCallable.java 977e1000c33fd50c3074d535076772ad6ff1c746
> giraph-core/src/test/java/org/apache/giraph/comm/RequestTest.java 70165729d5230a2620465ffbc69affc1896e1431
> giraph-core/src/test/java/org/apache/giraph/graph/TestVertexAndEdges.java d0a6c46114b5d2b42a5fcdba93e293d404e4257f
> giraph-core/src/test/java/org/apache/giraph/io/TestAdjacencyListTextVertexOutputFormat.java fdfb5e91dc692283a67f7fcaf309e8387f75a36d
> giraph-core/src/test/java/org/apache/giraph/io/TestLongDoubleDoubleAdjacencyListVertexInputFormat.java c67a1c44eed7b35a3c872d2032bc4e5ae77e7ef7
> giraph-core/src/test/java/org/apache/giraph/io/TestTextDoubleDoubleAdjacencyListVertexInputFormat.java 80340528326d9b37a380d9a761115b28b879ce2d
> giraph-core/src/test/java/org/apache/giraph/partition/TestGiraphTransferRegulator.java 8a1e9edaea53476d50dd90d70afc80c76d7f090a
> giraph-core/src/test/java/org/apache/giraph/partition/TestPartitionStores.java 45542b9083f8821ee664487791a7b6a50c115c8e
> giraph-core/src/test/java/org/apache/giraph/utils/MockUtils.java d210928374b21e42bac960c2d191386df42c119e
> giraph-examples/src/main/java/org/apache/giraph/examples/LongDoubleDoubleTextInputFormat.java f56b4f6a4c3f26e521829b5da4718ed8bc4e7b64
> giraph-examples/src/main/java/org/apache/giraph/examples/LongDoubleNullTextInputFormat.java bfb5f40bd2e7b8da321b5b818e377fea16fccecd
> giraph-examples/src/main/java/org/apache/giraph/examples/NormalizingLongDoubleDoubleTextInputFormat.java 5023a4e2ca80cf6fe0afe83598c11b00308fc630
> giraph-examples/src/main/java/org/apache/giraph/examples/SimplePageRankComputation.java d053bb3ceb0c77a3a6dbe46ffa330e9d5f8d035c
> giraph-examples/src/main/java/org/apache/giraph/examples/SimpleSuperstepComputation.java c3fd215b29a445b540a39b99f39cd7eaee195c91
> giraph-examples/src/test/java/org/apache/giraph/TestBspBasic.java 28edbba58a0513a2fe0b051e02e723dfd283de20
> giraph-examples/src/test/java/org/apache/giraph/examples/ConnectedComponentsComputationTestInMemory.java b94f8dcbd961d24f341cff252512c3a102b37ddb
> giraph-examples/src/test/java/org/apache/giraph/examples/SimpleShortestPathsComputationTest.java 93a3248f7fd5a94a84c8daf7c277ca954c859071
> giraph-examples/src/test/java/org/apache/giraph/examples/SimpleTriangleClosingComputationTest.java 73516a473ffd500500aeba083889ca3e5b7c3f03
> giraph-hbase/src/test/java/org/apache/giraph/io/hbase/edgemarker/TableEdgeInputFormat.java 1af46d1d6f9e2add4eb25ecdf1e5b351b5bf9132
> giraph-hbase/src/test/java/org/apache/giraph/io/hbase/edgemarker/TableEdgeOutputFormat.java a04d3864334158781b051e29fa8d02e3eba2b630
> giraph-hcatalog/src/main/java/org/apache/giraph/io/hcatalog/HCatalogVertexInputFormat.java 53c7eed811e326980f8adf4a34ba41a4152b5416
> giraph-hive/src/main/java/org/apache/giraph/hive/input/vertex/HiveToVertex.java a9736c69162e5d4a0a297cdea93b3e8186cdde5a
> giraph-hive/src/main/java/org/apache/giraph/hive/input/vertex/HiveVertexReader.java db61afff4fffe03c90ed100da2edf5ee28c98c4e
> giraph-hive/src/main/java/org/apache/giraph/hive/input/vertex/SimpleHiveToVertex.java 5483a6826d945a13a95dbd42eb8fa3e3aaeb2e97
>
> Diff: https://reviews.apache.org/r/11682/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Nitay Joffe
>
>
Re: Review Request: GIRAPH-468: Make Vertex an Interface (nitay)
Posted by Nitay Joffe <ni...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11682/
-----------------------------------------------------------
(Updated June 10, 2013, 8:34 p.m.)
Review request for giraph.
Description
-------
Vertex interface
This addresses bug GIRAPH-468.
https://issues.apache.org/jira/browse/GIRAPH-468
Diffs
-----
giraph-accumulo/src/test/java/org/apache/giraph/io/accumulo/edgemarker/AccumuloEdgeInputFormat.java 108ae61d5e57ba293a414028e1cb8b8c685b150c
giraph-core/src/main/java/org/apache/giraph/comm/SendMutationsCache.java 9348e61cd83985d920d3ed825d0fbdb3f3ab9fff
giraph-core/src/main/java/org/apache/giraph/comm/SendPartitionCache.java 68ce0955d8088ed87a6599fd5cd144cd7e470e6c
giraph-core/src/main/java/org/apache/giraph/comm/WorkerClientRequestProcessor.java 731d0eee0123ea9b5cecc47cff18d6be174f3e47
giraph-core/src/main/java/org/apache/giraph/comm/netty/NettyWorkerClientRequestProcessor.java 06965ca488aceb593a249a7c88bd49d660080c20
giraph-core/src/main/java/org/apache/giraph/conf/ImmutableClassesGiraphConfiguration.java aa5249875de551a341a4baac894c68553cbd6e63
giraph-core/src/main/java/org/apache/giraph/edge/EdgeStore.java 420bf9353871540594309ee9b4c129ae91ea7680
giraph-core/src/main/java/org/apache/giraph/graph/DefaultVertex.java PRE-CREATION
giraph-core/src/main/java/org/apache/giraph/graph/Vertex.java 1241ae757b29cb7bec80fd3bb3d2bab1ac9c21bd
giraph-core/src/main/java/org/apache/giraph/graph/VertexMutations.java 6f54dc701a5f5dc4589140dfb4a6d0dc5f128dca
giraph-core/src/main/java/org/apache/giraph/io/VertexReader.java 9695169b5ccf8325987e42de90e7cd3011c0b901
giraph-core/src/main/java/org/apache/giraph/io/VertexValueReader.java 70e721ec8e4a120a094206e5b857b9bb75eb7acd
giraph-core/src/main/java/org/apache/giraph/io/formats/JsonLongDoubleFloatDoubleVertexInputFormat.java 112860c12d203b734093973c3b5d9dd6db39b14a
giraph-core/src/main/java/org/apache/giraph/io/formats/PseudoRandomVertexInputFormat.java 91a19e6484631e4334b6c3843ba7aedfec169276
giraph-core/src/main/java/org/apache/giraph/io/formats/SequenceFileVertexInputFormat.java 1d31f4fb7a688ce314be5608f88726daa9152833
giraph-core/src/main/java/org/apache/giraph/io/formats/TextVertexInputFormat.java debdcccaf96cf105b5c0b3a73515f82860913178
giraph-core/src/main/java/org/apache/giraph/io/iterables/VertexReaderWrapper.java ca35c517666910e613431a3641f48f8df223f8b4
giraph-core/src/main/java/org/apache/giraph/partition/DiskBackedPartitionStore.java dadce0f14fe4ecac1dd35c35adf593a63e9583d9
giraph-core/src/main/java/org/apache/giraph/partition/Partition.java aebd34344b10cb3752c74c27f62bf3cfa5d3d565
giraph-core/src/main/java/org/apache/giraph/utils/InMemoryVertexInputFormat.java 6ca488cb42efec02b49f1ec9b4d22ec0b40bca42
giraph-core/src/main/java/org/apache/giraph/utils/TestGraph.java 6e46a76b656543c97a6b0c5eaad848633bd48bb6
giraph-core/src/test/java/org/apache/giraph/graph/TestVertexAndEdges.java d0a6c46114b5d2b42a5fcdba93e293d404e4257f
giraph-core/src/test/java/org/apache/giraph/io/TestAdjacencyListTextVertexOutputFormat.java fdfb5e91dc692283a67f7fcaf309e8387f75a36d
giraph-core/src/test/java/org/apache/giraph/io/TestLongDoubleDoubleAdjacencyListVertexInputFormat.java c67a1c44eed7b35a3c872d2032bc4e5ae77e7ef7
giraph-core/src/test/java/org/apache/giraph/io/TestTextDoubleDoubleAdjacencyListVertexInputFormat.java 80340528326d9b37a380d9a761115b28b879ce2d
giraph-core/src/test/java/org/apache/giraph/partition/TestGiraphTransferRegulator.java 8a1e9edaea53476d50dd90d70afc80c76d7f090a
giraph-core/src/test/java/org/apache/giraph/partition/TestPartitionStores.java 45542b9083f8821ee664487791a7b6a50c115c8e
giraph-core/src/test/java/org/apache/giraph/utils/MockUtils.java d210928374b21e42bac960c2d191386df42c119e
giraph-examples/src/main/java/org/apache/giraph/examples/LongDoubleDoubleTextInputFormat.java f56b4f6a4c3f26e521829b5da4718ed8bc4e7b64
giraph-examples/src/main/java/org/apache/giraph/examples/LongDoubleNullTextInputFormat.java bfb5f40bd2e7b8da321b5b818e377fea16fccecd
giraph-examples/src/main/java/org/apache/giraph/examples/NormalizingLongDoubleDoubleTextInputFormat.java 5023a4e2ca80cf6fe0afe83598c11b00308fc630
giraph-examples/src/main/java/org/apache/giraph/examples/SimplePageRankComputation.java d053bb3ceb0c77a3a6dbe46ffa330e9d5f8d035c
giraph-examples/src/main/java/org/apache/giraph/examples/SimpleSuperstepComputation.java c3fd215b29a445b540a39b99f39cd7eaee195c91
giraph-examples/src/test/java/org/apache/giraph/TestBspBasic.java 28edbba58a0513a2fe0b051e02e723dfd283de20
giraph-examples/src/test/java/org/apache/giraph/examples/ConnectedComponentsComputationTestInMemory.java b94f8dcbd961d24f341cff252512c3a102b37ddb
giraph-examples/src/test/java/org/apache/giraph/examples/SimpleShortestPathsComputationTest.java 93a3248f7fd5a94a84c8daf7c277ca954c859071
giraph-examples/src/test/java/org/apache/giraph/examples/SimpleTriangleClosingComputationTest.java 73516a473ffd500500aeba083889ca3e5b7c3f03
giraph-hbase/src/test/java/org/apache/giraph/io/hbase/edgemarker/TableEdgeInputFormat.java 1af46d1d6f9e2add4eb25ecdf1e5b351b5bf9132
giraph-hbase/src/test/java/org/apache/giraph/io/hbase/edgemarker/TableEdgeOutputFormat.java a04d3864334158781b051e29fa8d02e3eba2b630
giraph-hcatalog/src/main/java/org/apache/giraph/io/hcatalog/HCatalogVertexInputFormat.java 53c7eed811e326980f8adf4a34ba41a4152b5416
Diff: https://reviews.apache.org/r/11682/diff/
Testing
-------
Thanks,
Nitay Joffe
Re: Review Request: GIRAPH-468: Make Vertex an Interface (nitay)
Posted by Maja Kabiljo <ma...@fb.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11682/#review21667
-----------------------------------------------------------
Ship it!
Cool, +1
giraph-examples/src/main/java/org/apache/giraph/examples/SimpleSuperstepComputation.java
<https://reviews.apache.org/r/11682/#comment44783>
Leftover from previous patch
- Maja Kabiljo
On June 10, 2013, 8:12 p.m., Nitay Joffe wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/11682/
> -----------------------------------------------------------
>
> (Updated June 10, 2013, 8:12 p.m.)
>
>
> Review request for giraph.
>
>
> Description
> -------
>
> Vertex interface
>
>
> Diffs
> -----
>
> giraph-accumulo/src/test/java/org/apache/giraph/io/accumulo/edgemarker/AccumuloEdgeInputFormat.java 108ae61d5e57ba293a414028e1cb8b8c685b150c
> giraph-core/src/main/java/org/apache/giraph/comm/SendMutationsCache.java 9348e61cd83985d920d3ed825d0fbdb3f3ab9fff
> giraph-core/src/main/java/org/apache/giraph/comm/SendPartitionCache.java 68ce0955d8088ed87a6599fd5cd144cd7e470e6c
> giraph-core/src/main/java/org/apache/giraph/comm/WorkerClientRequestProcessor.java 731d0eee0123ea9b5cecc47cff18d6be174f3e47
> giraph-core/src/main/java/org/apache/giraph/comm/netty/NettyWorkerClientRequestProcessor.java 06965ca488aceb593a249a7c88bd49d660080c20
> giraph-core/src/main/java/org/apache/giraph/conf/ImmutableClassesGiraphConfiguration.java aa5249875de551a341a4baac894c68553cbd6e63
> giraph-core/src/main/java/org/apache/giraph/edge/EdgeStore.java 420bf9353871540594309ee9b4c129ae91ea7680
> giraph-core/src/main/java/org/apache/giraph/graph/DefaultVertex.java PRE-CREATION
> giraph-core/src/main/java/org/apache/giraph/graph/Vertex.java 1241ae757b29cb7bec80fd3bb3d2bab1ac9c21bd
> giraph-core/src/main/java/org/apache/giraph/graph/VertexMutations.java 6f54dc701a5f5dc4589140dfb4a6d0dc5f128dca
> giraph-core/src/main/java/org/apache/giraph/io/VertexReader.java 9695169b5ccf8325987e42de90e7cd3011c0b901
> giraph-core/src/main/java/org/apache/giraph/io/VertexValueReader.java 70e721ec8e4a120a094206e5b857b9bb75eb7acd
> giraph-core/src/main/java/org/apache/giraph/io/formats/JsonLongDoubleFloatDoubleVertexInputFormat.java 112860c12d203b734093973c3b5d9dd6db39b14a
> giraph-core/src/main/java/org/apache/giraph/io/formats/PseudoRandomVertexInputFormat.java 91a19e6484631e4334b6c3843ba7aedfec169276
> giraph-core/src/main/java/org/apache/giraph/io/formats/SequenceFileVertexInputFormat.java 1d31f4fb7a688ce314be5608f88726daa9152833
> giraph-core/src/main/java/org/apache/giraph/io/formats/TextVertexInputFormat.java debdcccaf96cf105b5c0b3a73515f82860913178
> giraph-core/src/main/java/org/apache/giraph/io/iterables/VertexReaderWrapper.java ca35c517666910e613431a3641f48f8df223f8b4
> giraph-core/src/main/java/org/apache/giraph/partition/DiskBackedPartitionStore.java dadce0f14fe4ecac1dd35c35adf593a63e9583d9
> giraph-core/src/main/java/org/apache/giraph/partition/Partition.java aebd34344b10cb3752c74c27f62bf3cfa5d3d565
> giraph-core/src/main/java/org/apache/giraph/utils/InMemoryVertexInputFormat.java 6ca488cb42efec02b49f1ec9b4d22ec0b40bca42
> giraph-core/src/main/java/org/apache/giraph/utils/TestGraph.java 6e46a76b656543c97a6b0c5eaad848633bd48bb6
> giraph-core/src/test/java/org/apache/giraph/graph/TestVertexAndEdges.java d0a6c46114b5d2b42a5fcdba93e293d404e4257f
> giraph-core/src/test/java/org/apache/giraph/io/TestAdjacencyListTextVertexOutputFormat.java fdfb5e91dc692283a67f7fcaf309e8387f75a36d
> giraph-core/src/test/java/org/apache/giraph/io/TestLongDoubleDoubleAdjacencyListVertexInputFormat.java c67a1c44eed7b35a3c872d2032bc4e5ae77e7ef7
> giraph-core/src/test/java/org/apache/giraph/io/TestTextDoubleDoubleAdjacencyListVertexInputFormat.java 80340528326d9b37a380d9a761115b28b879ce2d
> giraph-core/src/test/java/org/apache/giraph/partition/TestGiraphTransferRegulator.java 8a1e9edaea53476d50dd90d70afc80c76d7f090a
> giraph-core/src/test/java/org/apache/giraph/partition/TestPartitionStores.java 45542b9083f8821ee664487791a7b6a50c115c8e
> giraph-core/src/test/java/org/apache/giraph/utils/MockUtils.java d210928374b21e42bac960c2d191386df42c119e
> giraph-examples/src/main/java/org/apache/giraph/examples/LongDoubleDoubleTextInputFormat.java f56b4f6a4c3f26e521829b5da4718ed8bc4e7b64
> giraph-examples/src/main/java/org/apache/giraph/examples/LongDoubleNullTextInputFormat.java bfb5f40bd2e7b8da321b5b818e377fea16fccecd
> giraph-examples/src/main/java/org/apache/giraph/examples/NormalizingLongDoubleDoubleTextInputFormat.java 5023a4e2ca80cf6fe0afe83598c11b00308fc630
> giraph-examples/src/main/java/org/apache/giraph/examples/SimplePageRankComputation.java d053bb3ceb0c77a3a6dbe46ffa330e9d5f8d035c
> giraph-examples/src/main/java/org/apache/giraph/examples/SimpleSuperstepComputation.java c3fd215b29a445b540a39b99f39cd7eaee195c91
> giraph-examples/src/test/java/org/apache/giraph/TestBspBasic.java 28edbba58a0513a2fe0b051e02e723dfd283de20
> giraph-examples/src/test/java/org/apache/giraph/examples/ConnectedComponentsComputationTestInMemory.java b94f8dcbd961d24f341cff252512c3a102b37ddb
> giraph-examples/src/test/java/org/apache/giraph/examples/SimpleShortestPathsComputationTest.java 93a3248f7fd5a94a84c8daf7c277ca954c859071
> giraph-examples/src/test/java/org/apache/giraph/examples/SimpleTriangleClosingComputationTest.java 73516a473ffd500500aeba083889ca3e5b7c3f03
> giraph-hbase/src/test/java/org/apache/giraph/io/hbase/edgemarker/TableEdgeInputFormat.java 1af46d1d6f9e2add4eb25ecdf1e5b351b5bf9132
> giraph-hbase/src/test/java/org/apache/giraph/io/hbase/edgemarker/TableEdgeOutputFormat.java a04d3864334158781b051e29fa8d02e3eba2b630
> giraph-hcatalog/src/main/java/org/apache/giraph/io/hcatalog/HCatalogVertexInputFormat.java 53c7eed811e326980f8adf4a34ba41a4152b5416
>
> Diff: https://reviews.apache.org/r/11682/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Nitay Joffe
>
>
Re: Review Request: GIRAPH-468: Make Vertex an Interface (nitay)
Posted by Nitay Joffe <ni...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11682/
-----------------------------------------------------------
(Updated June 10, 2013, 8:12 p.m.)
Review request for giraph.
Description
-------
Vertex interface
Diffs (updated)
-----
giraph-accumulo/src/test/java/org/apache/giraph/io/accumulo/edgemarker/AccumuloEdgeInputFormat.java 108ae61d5e57ba293a414028e1cb8b8c685b150c
giraph-core/src/main/java/org/apache/giraph/comm/SendMutationsCache.java 9348e61cd83985d920d3ed825d0fbdb3f3ab9fff
giraph-core/src/main/java/org/apache/giraph/comm/SendPartitionCache.java 68ce0955d8088ed87a6599fd5cd144cd7e470e6c
giraph-core/src/main/java/org/apache/giraph/comm/WorkerClientRequestProcessor.java 731d0eee0123ea9b5cecc47cff18d6be174f3e47
giraph-core/src/main/java/org/apache/giraph/comm/netty/NettyWorkerClientRequestProcessor.java 06965ca488aceb593a249a7c88bd49d660080c20
giraph-core/src/main/java/org/apache/giraph/conf/ImmutableClassesGiraphConfiguration.java aa5249875de551a341a4baac894c68553cbd6e63
giraph-core/src/main/java/org/apache/giraph/edge/EdgeStore.java 420bf9353871540594309ee9b4c129ae91ea7680
giraph-core/src/main/java/org/apache/giraph/graph/DefaultVertex.java PRE-CREATION
giraph-core/src/main/java/org/apache/giraph/graph/Vertex.java 1241ae757b29cb7bec80fd3bb3d2bab1ac9c21bd
giraph-core/src/main/java/org/apache/giraph/graph/VertexMutations.java 6f54dc701a5f5dc4589140dfb4a6d0dc5f128dca
giraph-core/src/main/java/org/apache/giraph/io/VertexReader.java 9695169b5ccf8325987e42de90e7cd3011c0b901
giraph-core/src/main/java/org/apache/giraph/io/VertexValueReader.java 70e721ec8e4a120a094206e5b857b9bb75eb7acd
giraph-core/src/main/java/org/apache/giraph/io/formats/JsonLongDoubleFloatDoubleVertexInputFormat.java 112860c12d203b734093973c3b5d9dd6db39b14a
giraph-core/src/main/java/org/apache/giraph/io/formats/PseudoRandomVertexInputFormat.java 91a19e6484631e4334b6c3843ba7aedfec169276
giraph-core/src/main/java/org/apache/giraph/io/formats/SequenceFileVertexInputFormat.java 1d31f4fb7a688ce314be5608f88726daa9152833
giraph-core/src/main/java/org/apache/giraph/io/formats/TextVertexInputFormat.java debdcccaf96cf105b5c0b3a73515f82860913178
giraph-core/src/main/java/org/apache/giraph/io/iterables/VertexReaderWrapper.java ca35c517666910e613431a3641f48f8df223f8b4
giraph-core/src/main/java/org/apache/giraph/partition/DiskBackedPartitionStore.java dadce0f14fe4ecac1dd35c35adf593a63e9583d9
giraph-core/src/main/java/org/apache/giraph/partition/Partition.java aebd34344b10cb3752c74c27f62bf3cfa5d3d565
giraph-core/src/main/java/org/apache/giraph/utils/InMemoryVertexInputFormat.java 6ca488cb42efec02b49f1ec9b4d22ec0b40bca42
giraph-core/src/main/java/org/apache/giraph/utils/TestGraph.java 6e46a76b656543c97a6b0c5eaad848633bd48bb6
giraph-core/src/test/java/org/apache/giraph/graph/TestVertexAndEdges.java d0a6c46114b5d2b42a5fcdba93e293d404e4257f
giraph-core/src/test/java/org/apache/giraph/io/TestAdjacencyListTextVertexOutputFormat.java fdfb5e91dc692283a67f7fcaf309e8387f75a36d
giraph-core/src/test/java/org/apache/giraph/io/TestLongDoubleDoubleAdjacencyListVertexInputFormat.java c67a1c44eed7b35a3c872d2032bc4e5ae77e7ef7
giraph-core/src/test/java/org/apache/giraph/io/TestTextDoubleDoubleAdjacencyListVertexInputFormat.java 80340528326d9b37a380d9a761115b28b879ce2d
giraph-core/src/test/java/org/apache/giraph/partition/TestGiraphTransferRegulator.java 8a1e9edaea53476d50dd90d70afc80c76d7f090a
giraph-core/src/test/java/org/apache/giraph/partition/TestPartitionStores.java 45542b9083f8821ee664487791a7b6a50c115c8e
giraph-core/src/test/java/org/apache/giraph/utils/MockUtils.java d210928374b21e42bac960c2d191386df42c119e
giraph-examples/src/main/java/org/apache/giraph/examples/LongDoubleDoubleTextInputFormat.java f56b4f6a4c3f26e521829b5da4718ed8bc4e7b64
giraph-examples/src/main/java/org/apache/giraph/examples/LongDoubleNullTextInputFormat.java bfb5f40bd2e7b8da321b5b818e377fea16fccecd
giraph-examples/src/main/java/org/apache/giraph/examples/NormalizingLongDoubleDoubleTextInputFormat.java 5023a4e2ca80cf6fe0afe83598c11b00308fc630
giraph-examples/src/main/java/org/apache/giraph/examples/SimplePageRankComputation.java d053bb3ceb0c77a3a6dbe46ffa330e9d5f8d035c
giraph-examples/src/main/java/org/apache/giraph/examples/SimpleSuperstepComputation.java c3fd215b29a445b540a39b99f39cd7eaee195c91
giraph-examples/src/test/java/org/apache/giraph/TestBspBasic.java 28edbba58a0513a2fe0b051e02e723dfd283de20
giraph-examples/src/test/java/org/apache/giraph/examples/ConnectedComponentsComputationTestInMemory.java b94f8dcbd961d24f341cff252512c3a102b37ddb
giraph-examples/src/test/java/org/apache/giraph/examples/SimpleShortestPathsComputationTest.java 93a3248f7fd5a94a84c8daf7c277ca954c859071
giraph-examples/src/test/java/org/apache/giraph/examples/SimpleTriangleClosingComputationTest.java 73516a473ffd500500aeba083889ca3e5b7c3f03
giraph-hbase/src/test/java/org/apache/giraph/io/hbase/edgemarker/TableEdgeInputFormat.java 1af46d1d6f9e2add4eb25ecdf1e5b351b5bf9132
giraph-hbase/src/test/java/org/apache/giraph/io/hbase/edgemarker/TableEdgeOutputFormat.java a04d3864334158781b051e29fa8d02e3eba2b630
giraph-hcatalog/src/main/java/org/apache/giraph/io/hcatalog/HCatalogVertexInputFormat.java 53c7eed811e326980f8adf4a34ba41a4152b5416
Diff: https://reviews.apache.org/r/11682/diff/
Testing
-------
Thanks,
Nitay Joffe