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

(Updated March 27, 2013, 11:03 p.m.)


Review request for giraph.


Summary (updated)
-----------------

GIRAPH-587: Refactor configuration options


Description
-------

See https://issues.apache.org/jira/browse/GIRAPH-587


Diffs
-----

  checkstyle.xml 3d8a6d41388f97cbc50fc76d66e286c4d5a0bf1a 
  giraph-core/src/main/java/org/apache/giraph/benchmark/PageRankBenchmark.java 
2902fa9d321350364fedf481c95829cd9406db00 
  
giraph-core/src/main/java/org/apache/giraph/benchmark/ShortestPathsBenchmark.java
 1753f4ff9146fe598d209886c6f3e3087b204261 
  giraph-core/src/main/java/org/apache/giraph/bsp/BspInputFormat.java 
bce84b1d839f08eb6e569fb59e52dd7966e9bbb5 
  giraph-core/src/main/java/org/apache/giraph/bsp/BspService.java 
969e2a5089279bddbdc75be847b681f6f1fda25a 
  giraph-core/src/main/java/org/apache/giraph/comm/SendEdgeCache.java 
679cf6fffb97a526fd8f122c9f73399bd81b28f9 
  giraph-core/src/main/java/org/apache/giraph/comm/SendMessageCache.java 
07dc380a4bc1dd50ce12d6c06bea4f6c71bf6a1b 
  giraph-core/src/main/java/org/apache/giraph/comm/ServerData.java 
70dc156adfc804d9a9af6442543e1555c3687a27 
  
giraph-core/src/main/java/org/apache/giraph/comm/messages/SequentialFileMessageStore.java
 bdc543571c7a40cb28c5096f41ea12951ecdca9d 
  giraph-core/src/main/java/org/apache/giraph/comm/netty/NettyClient.java 
af76410f0ccf512d63939398d4afacbc0cdc449a 
  giraph-core/src/main/java/org/apache/giraph/comm/netty/NettyServer.java 
f31dd4af0d06dd48e6dbe401e3c60860e83789e9 
  
giraph-core/src/main/java/org/apache/giraph/comm/netty/NettyWorkerClientRequestProcessor.java
 e58030e4b6d016eef47bf46fdc0d4eccec93826f 
  giraph-core/src/main/java/org/apache/giraph/comm/netty/NettyWorkerServer.java 
1fb0580325ebae880fdbed94925edad70ca7f750 
  
giraph-core/src/main/java/org/apache/giraph/comm/netty/handler/RequestEncoder.java
 4e739cb1a32d8e1c2869d701845a57029f2ba238 
  
giraph-core/src/main/java/org/apache/giraph/comm/netty/handler/RequestServerHandler.java
 bbf31c78ca539287fcda50ba2f062f9f86e8d682 
  
giraph-core/src/main/java/org/apache/giraph/comm/netty/handler/ResponseClientHandler.java
 1803be4820347a5c92f5a052fc16bb90f588e2e2 
  
giraph-core/src/main/java/org/apache/giraph/comm/netty/handler/SaslServerHandler.java
 d06fd09a00fbd4e0799daadf151ab58ff58478e5 
  giraph-core/src/main/java/org/apache/giraph/conf/AbstractConfOption.java 
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/conf/AllOptions.java PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/conf/BooleanConfOption.java 
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/conf/ClassConfOption.java 
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/conf/ConfOptionType.java 
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/conf/FloatConfOption.java 
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/conf/GiraphClasses.java 
c13f3a268edc0d514314d8813ce0e59dd4ed056f 
  giraph-core/src/main/java/org/apache/giraph/conf/GiraphConfiguration.java 
ffcae6e79cf0bd96745ef9bc9fc420f5b28494b2 
  giraph-core/src/main/java/org/apache/giraph/conf/GiraphConstants.java 
7882d06fddd70eb9fef7a8d0463c6618b39d3d4d 
  
giraph-core/src/main/java/org/apache/giraph/conf/ImmutableClassesGiraphConfiguration.java
 de85ab63306b63145159cfaf3d587fcef8ae58ce 
  giraph-core/src/main/java/org/apache/giraph/conf/IntConfOption.java 
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/conf/LongConfOption.java 
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/conf/StrConfOption.java 
PRE-CREATION 
  giraph-core/src/main/java/org/apache/giraph/graph/GraphTaskManager.java 
e74c59a91b80ac69e0aa53b04a3669329a87ac2f 
  
giraph-core/src/main/java/org/apache/giraph/job/GiraphConfigurationValidator.java
 1e0577373e6627d35976c6c2d4b2264b3546e62f 
  giraph-core/src/main/java/org/apache/giraph/job/GiraphJob.java 
da85d1cfd5d5b42e9c3173f10a466c6b03b941b4 
  giraph-core/src/main/java/org/apache/giraph/master/BspServiceMaster.java 
6c979d6aaf78000521083f5374c56341867551aa 
  giraph-core/src/main/java/org/apache/giraph/master/MasterThread.java 
40c6b740315b40b68e9300991869e6b0a203b2ca 
  
giraph-core/src/main/java/org/apache/giraph/partition/DiskBackedPartitionStore.java
 6bc95917efe4acdaca19f4a390e8dcb9824d7675 
  giraph-core/src/main/java/org/apache/giraph/partition/PartitionUtils.java 
c83ca456cbaf6bba26421f4f766b07cc35665cdd 
  giraph-core/src/main/java/org/apache/giraph/partition/SimplePartition.java 
ae8556fd47fa1d2c1a8fd0c0b7fb4bb95215c044 
  giraph-core/src/main/java/org/apache/giraph/utils/InternalVertexRunner.java 
2bba6724cd5dd4aaa5d94b0cefdefcd7fdb4fa7c 
  giraph-core/src/main/java/org/apache/giraph/utils/JMapHistoDumper.java 
463510fb1fe4b54fe4469cec43eacb5b1198f768 
  giraph-core/src/main/java/org/apache/giraph/zk/GiraphZooKeeperAdmin.java 
7589a0908e720570812d658612dfa5617ffad828 
  giraph-core/src/main/java/org/apache/giraph/zk/ZooKeeperManager.java 
add57fcaaf15ff5400df1ce700851f9356296f81 
  giraph-core/src/test/java/org/apache/giraph/BspCase.java 
a6eef20429508184f0b8cb050f74553720022467 
  giraph-core/src/test/java/org/apache/giraph/comm/RequestFailureTest.java 
cc7cb171333509dfbca43dd3d0361fb5ed13537f 
  giraph-core/src/test/java/org/apache/giraph/comm/RequestTest.java 
c8d6b3b7667b9dfe6e6dabe904b37cf8be7ee121 
  giraph-core/src/test/java/org/apache/giraph/comm/SaslConnectionTest.java 
3094f3e64a246ae5c9ec8cda8b8ef7d7a4605f05 
  giraph-core/src/test/java/org/apache/giraph/conf/TestGiraphConfiguration.java 
cd3719732e92a62d4f2507d60b6a7874e264943e 
  giraph-core/src/test/java/org/apache/giraph/conf/TestObjectCreation.java 
9f0920a5349ddc6a228035ea386fd55e799702a8 
  giraph-core/src/test/java/org/apache/giraph/master/TestMasterObserver.java 
54a8b92ff37780a61e1f615f764af897dba03988 
  
giraph-core/src/test/java/org/apache/giraph/partition/TestPartitionStores.java 
d403dd8e132be16e7d080f76b6b2b63fe574fc5a 
  giraph-examples/src/test/java/org/apache/giraph/TestAutoCheckpoint.java 
efbe320bd8ed96ad8d27c394d1ec8b182f385239 
  giraph-examples/src/test/java/org/apache/giraph/TestBspBasic.java 
5234730106949676732ae5ce4428ff8c65d023dd 
  giraph-examples/src/test/java/org/apache/giraph/TestManualCheckpoint.java 
eb2338c2dbafb2efdc2707f7fc323a26fcb3ea8f 
  giraph-examples/src/test/java/org/apache/giraph/TestPartitionContext.java 
41f5e3c841a89f5c5d500a414f6fc4ae804b8705 
  
giraph-examples/src/test/java/org/apache/giraph/aggregators/TestAggregatorsHandling.java
 7deeb42a71a884b1d62f2b6bcac04d2dd61960db 
  giraph-examples/src/test/java/org/apache/giraph/examples/TestPageRank.java 
f56d7e5f4e845ef685a4eaadbdad528e60f3e8a4 
  
giraph-examples/src/test/java/org/apache/giraph/examples/TryMultiIpcBindingPortsTest.java
 c5d3a552ace96a4378e2181c8bdd2d871e487692 
  giraph-examples/src/test/java/org/apache/giraph/vertex/TestVertexTypes.java 
5ca55b6e5ab84fafc6129590238fbbee37b3185d 

Diff: https://reviews.apache.org/r/10156/diff/


Testing
-------


Thanks,

Nitay Joffe

Reply via email to