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


+1

- Alessandro Presta


On July 30, 2013, 6:44 p.m., Nitay Joffe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12543/
> -----------------------------------------------------------
> 
> (Updated July 30, 2013, 6:44 p.m.)
> 
> 
> Review request for giraph.
> 
> 
> Bugs: GIRAPH-717
>     https://issues.apache.org/jira/browse/GIRAPH-717
> 
> 
> Repository: giraph-git
> 
> 
> Description
> -------
> 
> See JIRA
> 
> 
> Diffs
> -----
> 
>   findbugs-exclude.xml 2f7f4007cbfb45f077377328bd43b09ba0424af4 
>   giraph-core/pom.xml cab0157803d130ab5ab342c6c5c6d5744f8bdaa3 
>   
> giraph-core/src/main/java/org/apache/giraph/comm/messages/ByteArrayMessagesPerVertexStore.java
>  ba6d020726f3609c7cc3a9469e76058b787c576f 
>   
> giraph-core/src/main/java/org/apache/giraph/comm/messages/InMemoryMessageStoreFactory.java
>  40674e1c74ede0c39a5a55f5eb0e2f114aecb481 
>   
> giraph-core/src/main/java/org/apache/giraph/comm/messages/MessagesIterable.java
>  f3f327ffdfe4ec68cb5a61a3bd16f0cf82bd7df9 
>   
> giraph-core/src/main/java/org/apache/giraph/comm/messages/OneMessagePerVertexStore.java
>  29260dfac7ca03c274a10dd15deb194e3e41599b 
>   
> giraph-core/src/main/java/org/apache/giraph/comm/messages/out_of_core/SequentialFileMessageStore.java
>  1ff0e189cd954c98c594288a9b9ea98bee773960 
>   giraph-core/src/main/java/org/apache/giraph/conf/GiraphClasses.java 
> 66558342df172aa85e75ad4aaa1c8a93568843eb 
>   giraph-core/src/main/java/org/apache/giraph/conf/GiraphConfiguration.java 
> de0539b819b3d01afc240143ada2d7303203bef7 
>   giraph-core/src/main/java/org/apache/giraph/conf/GiraphConstants.java 
> f6233e329f189888f9369568bdd06f64c34c36fa 
>   
> giraph-core/src/main/java/org/apache/giraph/conf/ImmutableClassesGiraphConfiguration.java
>  db24423cdefb968f71fa2174a1b9788c945e6b24 
>   giraph-core/src/main/java/org/apache/giraph/conf/PerGraphTypeBoolean.java 
> PRE-CREATION 
>   
> giraph-core/src/main/java/org/apache/giraph/conf/PerGraphTypeBooleanConfOption.java
>  PRE-CREATION 
>   giraph-core/src/main/java/org/apache/giraph/conf/PerGraphTypeEnum.java 
> PRE-CREATION 
>   
> giraph-core/src/main/java/org/apache/giraph/conf/PerGraphTypeEnumConfOption.java
>  PRE-CREATION 
>   
> giraph-core/src/main/java/org/apache/giraph/factories/AbstractMessageValueFactory.java
>  f426d5f55e5ce3eee98bb8c3638ec9e253c1a26f 
>   
> giraph-core/src/main/java/org/apache/giraph/factories/DefaultEdgeValueFactory.java
>  55d27e781290c406011a2ae201a811f4d85e8f31 
>   
> giraph-core/src/main/java/org/apache/giraph/factories/DefaultVertexIdFactory.java
>  4977b44cc5fdc6ca27d789cf66b542ba177d12af 
>   
> giraph-core/src/main/java/org/apache/giraph/factories/DefaultVertexValueFactory.java
>  caf312f6f5507281ead0b9f94efacab124ac239c 
>   giraph-core/src/main/java/org/apache/giraph/factories/EdgeValueFactory.java 
> 751c6e61d653519a8f7438b1dfc103b86774705f 
>   
> giraph-core/src/main/java/org/apache/giraph/factories/MessageValueFactory.java
>  a95e1a27a1235cbd57ad48fc77ca1d183bb96075 
>   
> giraph-core/src/main/java/org/apache/giraph/factories/TestMessageValueFactory.java
>  3d376de6272f0c39702c2e174e6fc66f549721e0 
>   giraph-core/src/main/java/org/apache/giraph/factories/ValueFactory.java 
> PRE-CREATION 
>   giraph-core/src/main/java/org/apache/giraph/factories/ValueFactoryBase.java 
> f92fb747b0c1f6d8b3a87a9e2813c3f0bcae4ae8 
>   giraph-core/src/main/java/org/apache/giraph/factories/VertexIdFactory.java 
> 328bda8a94a8b8f6aad43dabaadea263cc503650 
>   
> giraph-core/src/main/java/org/apache/giraph/factories/VertexValueFactory.java 
> d2f62ab99d6cf2d26b1a2fa829e2213ff4c7da5d 
>   giraph-core/src/main/java/org/apache/giraph/graph/AbstractComputation.java 
> PRE-CREATION 
>   giraph-core/src/main/java/org/apache/giraph/graph/BasicComputation.java 
> 180c5d34a7c02c37ff935941b02d50dd3018eff3 
>   giraph-core/src/main/java/org/apache/giraph/graph/Computation.java 
> 34b010980fd787a85622556e369624d278ed425a 
>   giraph-core/src/main/java/org/apache/giraph/graph/GraphType.java 
> PRE-CREATION 
>   giraph-core/src/main/java/org/apache/giraph/jython/JythonComputation.java 
> PRE-CREATION 
>   
> giraph-core/src/main/java/org/apache/giraph/jython/JythonComputationFactory.java
>  80e4e7620ff6953b167ef4ff81177fb9914a8893 
>   
> giraph-core/src/main/java/org/apache/giraph/jython/JythonGiraphComputation.java
>  PRE-CREATION 
>   giraph-core/src/main/java/org/apache/giraph/jython/JythonJob.java 
> PRE-CREATION 
>   giraph-core/src/main/java/org/apache/giraph/jython/JythonOptions.java 
> PRE-CREATION 
>   giraph-core/src/main/java/org/apache/giraph/jython/JythonUtils.java 
> e747e8475281b55acd75273a7b5b9ef1a256ef14 
>   
> giraph-core/src/main/java/org/apache/giraph/jython/factories/JythonComputationFactory.java
>  PRE-CREATION 
>   
> giraph-core/src/main/java/org/apache/giraph/jython/factories/JythonEdgeValueFactory.java
>  PRE-CREATION 
>   
> giraph-core/src/main/java/org/apache/giraph/jython/factories/JythonFactoryBase.java
>  PRE-CREATION 
>   
> giraph-core/src/main/java/org/apache/giraph/jython/factories/JythonIncomingMessageValueFactory.java
>  PRE-CREATION 
>   
> giraph-core/src/main/java/org/apache/giraph/jython/factories/JythonMessageValueFactory.java
>  PRE-CREATION 
>   
> giraph-core/src/main/java/org/apache/giraph/jython/factories/JythonOutgoingMessageValueFactory.java
>  PRE-CREATION 
>   
> giraph-core/src/main/java/org/apache/giraph/jython/factories/JythonVertexIdFactory.java
>  PRE-CREATION 
>   
> giraph-core/src/main/java/org/apache/giraph/jython/factories/JythonVertexValueFactory.java
>  PRE-CREATION 
>   
> giraph-core/src/main/java/org/apache/giraph/jython/factories/package-info.java
>  PRE-CREATION 
>   
> giraph-core/src/main/java/org/apache/giraph/jython/wrappers/JythonWrapperBase.java
>  PRE-CREATION 
>   
> giraph-core/src/main/java/org/apache/giraph/jython/wrappers/JythonWritableWrapper.java
>  PRE-CREATION 
>   
> giraph-core/src/main/java/org/apache/giraph/jython/wrappers/package-info.java 
> PRE-CREATION 
>   giraph-core/src/main/java/org/apache/giraph/master/MasterCompute.java 
> a3427dc3a8137357e3bb24a4a48398e8a64643b6 
>   giraph-core/src/main/java/org/apache/giraph/scripting/ScriptLoader.java 
> bf3e1520e951252deb2dbb51a20c840f6dffa031 
>   
> giraph-core/src/main/java/org/apache/giraph/types/BooleanToBooleanWritableWrapper.java
>  d0637504ce4d5ec001503d571cb1dd2b7d49e606 
>   
> giraph-core/src/main/java/org/apache/giraph/types/ByteToByteWritableWrapper.java
>  1b36178080fd69753e0b8c2e9f6a942e36e04014 
>   
> giraph-core/src/main/java/org/apache/giraph/types/ByteToIntWritableWrapper.java
>  adbbb98947b320ba4db97fe849f2a6bff3320898 
>   
> giraph-core/src/main/java/org/apache/giraph/types/ByteToLongWritableWrapper.java
>  a4f22a5d523858b5a8e63987bb43e448c768d7e4 
>   
> giraph-core/src/main/java/org/apache/giraph/types/DoubleToDoubleWritableWrapper.java
>  a1fae98d193ec8a77787ee587457a59dca121e72 
>   
> giraph-core/src/main/java/org/apache/giraph/types/FloatToDoubleWritableWrapper.java
>  d51edcab96fa927d61bdd7a8cfb6c92e99e4bf20 
>   
> giraph-core/src/main/java/org/apache/giraph/types/FloatToFloatWritableWrapper.java
>  4f4dd9796e308474cfbba39dcff7c0a718afbfd4 
>   
> giraph-core/src/main/java/org/apache/giraph/types/IntToIntWritableWrapper.java
>  4d868ac4cead60c19a3a1ee8ec0c7d1e43da736e 
>   
> giraph-core/src/main/java/org/apache/giraph/types/IntToLongWritableWrapper.java
>  9340d5222633c5ed57c25ef767fdb06a5735f208 
>   
> giraph-core/src/main/java/org/apache/giraph/types/LongToLongWritableWrapper.java
>  2211ccb926d71319b7fe5825c4b9fd590a1b7286 
>   
> giraph-core/src/main/java/org/apache/giraph/types/ShortToIntWritableWrapper.java
>  6df538ade7ceaad55fbb4a47f9eaf39c58432793 
>   
> giraph-core/src/main/java/org/apache/giraph/types/ShortToLongWritableWrapper.java
>  f3d56b5c77df25619ac21d52d74b3badbac72d86 
>   giraph-core/src/main/java/org/apache/giraph/types/WritableWrapper.java 
> edb62a5b91d6396f968c28fb3bf297ec46e44af4 
>   
> giraph-core/src/main/java/org/apache/giraph/utils/ByteArrayOneToAllMessages.java
>  8a6fc1225f2678aed29f62d299b626d18e9ce0c4 
>   
> giraph-core/src/main/java/org/apache/giraph/utils/ByteArrayVertexIdMessages.java
>  3c8afd025c85dd97e90831ea0de2837dfa406b31 
>   giraph-core/src/main/java/org/apache/giraph/utils/ConfigurationUtils.java 
> 0424ff541f9ee12feac0e370d71085f8d91b9019 
>   giraph-core/src/main/java/org/apache/giraph/utils/ReflectionUtils.java 
> 908376993c45c72bd531eabf6d5817d1aece197a 
>   giraph-core/src/test/java/org/apache/giraph/io/TestEdgeInput.java 
> 425fcba10ac8cd63e93520d6da0f064155f29d5e 
>   giraph-core/src/test/java/org/apache/giraph/jython/TestJython.java 
> b5110bb5996235bfb892c578bdaeb38d7bc6f2b5 
>   giraph-core/src/test/java/org/apache/giraph/jython/TestJythonBasic.java 
> PRE-CREATION 
>   
> giraph-core/src/test/java/org/apache/giraph/jython/TestJythonComputation.java 
> PRE-CREATION 
>   
> giraph-core/src/test/java/org/apache/giraph/jython/TestJythonWritableWrapper.java
>  PRE-CREATION 
>   
> giraph-core/src/test/java/org/apache/giraph/master/TestComputationCombinerTypes.java
>  f9bd4e42f5bfa479cad38bec820e8b21b633bacd 
>   giraph-core/src/test/java/org/apache/giraph/master/TestSwitchClasses.java 
> 4069972c747fc3be86d99f15aef21f99c49ad940 
>   giraph-core/src/test/java/org/apache/giraph/utils/TestReflectionUtils.java 
> 1c3eed3d46ccc9dbcd5ed100146a16216641677c 
>   giraph-core/src/test/resources/org/apache/giraph/jython/count-edges.py 
> 4cad945ba25eb9950d55a9aac12ebd062ad69175 
>   giraph-examples/src/test/java/org/apache/giraph/TestBspBasic.java 
> f9e77affacbb4890e2006383c37e3bb281cda0d7 
>   
> giraph-examples/src/test/java/org/apache/giraph/vertex/TestComputationTypes.java
>  fe354fadfe29b71bbd44158939116053e54683d5 
>   
> giraph-hcatalog/src/main/java/org/apache/giraph/io/hcatalog/HCatGiraphRunner.java
>  6135c7d1c4e816f2dbbf433910552219a26a5f32 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/column/HiveReadableColumn.java
>  PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/column/HiveWritableColumn.java
>  PRE-CREATION 
>   giraph-hive/src/main/java/org/apache/giraph/hive/column/package-info.java 
> PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/common/GiraphHiveConstants.java
>  e492694959d8cb488b4bc9d5a1bf4b321fe396f2 
>   giraph-hive/src/main/java/org/apache/giraph/hive/common/HiveUtils.java 
> ae2a402c69da105d7127ed2396c0d8e661d21be8 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/common/LanguageAndType.java 
> PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/input/edge/AbstractHiveToEdge.java
>  a3c323e3d8d964295ebfa0de6ca4fbbe52b9bbe2 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/input/edge/TypedHiveToEdge.java
>  ac62827efb5a8e0cc12c144ba2c5a31eb7018e40 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/input/vertex/AbstractHiveToVertex.java
>  45cc20978ff43b5bde5fca64f5c2718d27ecf5fc 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/input/vertex/TypedHiveToVertex.java
>  5ca7d34c49ba44b7626df900835e6d925c4480e6 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/jython/HiveJythonRunner.java 
> PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/jython/HiveJythonUtils.java 
> PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/jython/JythonColumnReader.java
>  PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/jython/JythonColumnWriter.java
>  PRE-CREATION 
>   giraph-hive/src/main/java/org/apache/giraph/hive/jython/JythonHiveIO.java 
> PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/jython/JythonHiveReader.java 
> PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/jython/JythonHiveToEdge.java 
> PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/jython/JythonHiveToVertex.java
>  PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/jython/JythonHiveWriter.java 
> PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/jython/JythonReadableColumn.java
>  PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/jython/JythonVertexToHive.java
>  PRE-CREATION 
>   giraph-hive/src/main/java/org/apache/giraph/hive/jython/package-info.java 
> PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/output/TypedVertexToHive.java
>  8d6fd9a4ca589291aae7c994955fe4bc845b199f 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/primitives/PrimitiveValueReader.java
>  PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/primitives/PrimitiveValueWriter.java
>  PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/primitives/package-info.java 
> PRE-CREATION 
>   giraph-hive/src/main/java/org/apache/giraph/hive/types/HiveValueReader.java 
> c0fad122f302a3cef4ffec8ad51ff116c8a8f51c 
>   giraph-hive/src/main/java/org/apache/giraph/hive/types/HiveValueWriter.java 
> f38f79a482c627cdd2f6797db684d83e2717b724 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/types/HiveVertexIdReader.java
>  1ad6f7f734bd34b424ee0f1d9536ee4e5c3579d5 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/types/HiveVertexIdWriter.java
>  33196f5bbfc9c10fc944ad14f14dbdb7c1f91fdd 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/types/TypedValueReader.java 
> e9b068732459f0d586187b7a86782859d44590ac 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/types/TypedValueWriter.java 
> 8841bda5e9400859776acd6df6b04c3fc4b97b13 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/types/TypedVertexIdReader.java
>  6d80d37082ffe929a5e00c3ae17ef57e200659b9 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/types/TypedVertexIdWriter.java
>  aece54b5715eb29cd220772dc5061266501eed11 
>   giraph-hive/src/main/java/org/apache/giraph/hive/types/package-info.java 
> 700463766f118e4a18ec8a8d2a9156adb16b2b31 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/values/HiveValueReader.java 
> PRE-CREATION 
>   
> giraph-hive/src/main/java/org/apache/giraph/hive/values/HiveValueWriter.java 
> PRE-CREATION 
>   giraph-hive/src/main/java/org/apache/giraph/hive/values/package-info.java 
> PRE-CREATION 
>   giraph-hive/src/test/java/org/apache/giraph/hive/GiraphHiveTestBase.java 
> 49a36d094869463ec4b90abc5e787ca5a4b80121 
>   giraph-hive/src/test/java/org/apache/giraph/hive/Helpers.java 
> 00c00ca5ecacb94517e4e7bf77d0f25630733a29 
>   
> giraph-hive/src/test/java/org/apache/giraph/hive/input/HiveEdgeInputTest.java 
> d5bbb954d096d15b720ff0b2590b75de67d776a2 
>   
> giraph-hive/src/test/java/org/apache/giraph/hive/input/HiveVertexInputTest.java
>  af850d5c028fe9528733a3054c217fa690712684 
>   
> giraph-hive/src/test/java/org/apache/giraph/hive/jython/TestHiveJythonComplexTypes.java
>  PRE-CREATION 
>   
> giraph-hive/src/test/java/org/apache/giraph/hive/jython/TestHiveJythonPrimitives.java
>  PRE-CREATION 
>   giraph-hive/src/test/java/org/apache/giraph/hive/output/HiveOutputTest.java 
> 4d4d976dd99ca52d4a9bd9e3fef4c29119143721 
>   
> giraph-hive/src/test/resources/org/apache/giraph/jython/count-edges-launcher.py
>  PRE-CREATION 
>   
> giraph-hive/src/test/resources/org/apache/giraph/jython/fake-label-propagation-launcher.py
>  PRE-CREATION 
>   
> giraph-hive/src/test/resources/org/apache/giraph/jython/fake-label-propagation-worker.py
>  PRE-CREATION 
>   pom.xml 8aa1f457646ce10045f7bed7ff541eb4381f9bac 
> 
> Diff: https://reviews.apache.org/r/12543/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Nitay Joffe
> 
>

Reply via email to