-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31392/
-----------------------------------------------------------
(Updated March 8, 2015, 7:39 p.m.)
Review request for samza.
Bugs: SAMZA-555
https://issues.apache.org/jira/browse/SAMZA-555
Repository: samza
Description (updated)
-------
merge master
don't npe if a container has no assignments
killing zk works
listen to existing containers when becoming leader
spelling mistake
fix comments
add notes
clear assignments after coordinator is elected.
add logging
make job runner block until job finishes
add standalone job and standalone job factory. make zk work with hierarchical
chroot paths.
add unsubscribe to coordinator controller
unsubscribe to assignment changes in container listener
both tests pass.
fix bug in container controller to announce ownership
add initial container controller
make test do asserts
standalone main method works as expected.
rework assignTasksToContainers. controller isn't watching /containers
sequential ids yet, but it should.
fiddling around with CLI test
writing a little main method to play with zk and coordinator
initial coordinator draft
add missing leadership check
working on zk coordinator controller.
messing around
sketch out a hacky controller.
adding core dependency
create samza-standalone project
Diffs (updated)
-----
build.gradle 0a268ac7a3819cf46b54a93e0e3171455371456a
gradle/dependency-versions.gradle 84be50b216e7dc3c430e0979a933d846f8ebbb8d
samza-core/src/main/scala/org/apache/samza/config/JobConfig.scala
3b6685e00837a4aaf809813e62b7e52823bc07a9
samza-core/src/main/scala/org/apache/samza/container/RunLoop.scala
499f5c6a58ff88ef9105b7b6ca168215fb82f35c
samza-core/src/main/scala/org/apache/samza/job/JobRunner.scala
0b720ec4dd83c71fd1ce5071571c7a10babf0ddc
samza-log4j/src/main/java/org/apache/samza/logging/log4j/serializers/LoggingEventStringSerde.java
8d8f5e8a8e5fd1e4d9e5482a5accd4a7ece463bc
samza-standalone/src/main/java/org/apache/samza/job/standalone/StandaloneJob.java
PRE-CREATION
samza-standalone/src/main/java/org/apache/samza/job/standalone/StandaloneJobFactory.java
PRE-CREATION
samza-standalone/src/main/java/org/apache/samza/job/standalone/controller/StandaloneZkContainerController.java
PRE-CREATION
samza-standalone/src/main/java/org/apache/samza/job/standalone/controller/StandaloneZkCoordinatorController.java
PRE-CREATION
samza-standalone/src/main/java/org/apache/samza/job/standalone/controller/StandaloneZkCoordinatorState.java
PRE-CREATION
samza-standalone/src/main/java/org/apache/samza/serializers/zk/ZkJsonSerde.java
PRE-CREATION
samza-standalone/src/main/java/org/apache/samza/util/ZkUtil.java PRE-CREATION
samza-test/src/main/java/org/apache/samza/system/mock/MockSystemConsumer.java
a282dbb2976cb916d41649c3fbe070008c6621ee
samza-test/src/main/java/org/apache/samza/task/MockTask.java PRE-CREATION
samza-test/src/test/java/org/apache/samza/job/standalone/controller/TestStandaloneZkCoordinatorController.java
PRE-CREATION
settings.gradle bb07a3b84b14dcef94da1bb166eab6aa3d0026bb
Diff: https://reviews.apache.org/r/31392/diff/
Testing
-------
Thanks,
Chris Riccomini