----------------------------------------------------------- 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 > >