----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16430/ -----------------------------------------------------------
(Updated Jan. 2, 2014, 11:05 p.m.) Review request for samza. Changes ------- Added ASF headers to all three mock classes, and the perf test class. Left systemProperties as-is. Reason for not forwarding all system properties is that the Gradle main-proc is setting a LOT of system properties (50+), and some of them were actually causing Jacoco to freak out in the test-proc. I wanted to isolate the risk of accidentally bleeding system properties into child-procs as much as possible. Repository: samza Description (updated) ------- adding asf headers to mock classes. adding apache header to perf test. fix some docs in perf test. fix scala 2.8.1 compile issue related to adding ints in SystemConsumers. revert accidental reviewboardrc checkin. adding one extra doc to build.gradle adding javadocs everywhere adding caching for system fetch map in SystemConsumers.poll bump up heap for performance test. move samza core to compile time dependency since mock system uses it. add mock system and test for performance. Diffs (updated) ----- .reviewboardrc 0ee6a71c84942b7a626ce66994e4abfcb87bf4c8 build.gradle 49a8459c43102b597c69ba6d7e0262baf843b513 samza-core/src/main/scala/org/apache/samza/system/SystemConsumers.scala 5cbffe5fdf9dd60d0b76aa88ab0fef09addb4e31 samza-test/java.hprof.txt c5693d20c2caa471f164b7cb05d0c32eee9e3b03 samza-test/src/main/java/org/apache/samza/system/mock/MockSystemAdmin.java PRE-CREATION samza-test/src/main/java/org/apache/samza/system/mock/MockSystemConsumer.java PRE-CREATION samza-test/src/main/java/org/apache/samza/system/mock/MockSystemFactory.java PRE-CREATION samza-test/src/test/scala/org/apache/samza/test/integration/TestStatefulTask.scala 9602a527a8fdbc8e342413eeb62d1aef27b1972d samza-test/src/test/scala/org/apache/samza/test/performance/TestSamzaContainerPerformance.scala PRE-CREATION Diff: https://reviews.apache.org/r/16430/diff/ Testing ------- Thanks, Chris Riccomini
