-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53163/
-----------------------------------------------------------
(Updated Oct. 27, 2016, 9:25 a.m.)
Review request for samza and Jagadish Venkatraman.
Bugs: SAMZA-901
https://issues.apache.org/jira/browse/SAMZA-901
Repository: samza
Description
-------
SAMZA-901 SamzaAppState re-design for thread safety
Diffs (updated)
-----
samza-core/src/main/java/org/apache/samza/clustermanager/AbstractContainerAllocator.java
d47f217
samza-core/src/main/java/org/apache/samza/clustermanager/ClusterBasedJobCoordinator.java
d0d4e34
samza-core/src/main/java/org/apache/samza/clustermanager/ContainerProcessManager.java
b4309d9
samza-core/src/main/java/org/apache/samza/clustermanager/HostAwareContainerAllocator.java
da73049
samza-core/src/main/java/org/apache/samza/clustermanager/SamzaApplicationState.java
cf91044
samza-core/src/main/scala/org/apache/samza/metrics/ContainerProcessManagerMetrics.scala
f24beb1
samza-core/src/test/java/org/apache/samza/clustermanager/TestContainerAllocator.java
5351bc3
samza-core/src/test/java/org/apache/samza/clustermanager/TestContainerProcessManager.java
0d61814
samza-core/src/test/java/org/apache/samza/clustermanager/TestHostAwareContainerAllocator.java
b6651f2
samza-yarn/src/main/java/org/apache/samza/job/yarn/YarnResourceManagerFactory.java
988a8e8
samza-yarn/src/main/resources/scalate/WEB-INF/views/index.scaml 93176ff
samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMasterMetrics.scala
8a5b4aa
samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaYarnAppMasterLifecycle.scala
c9c1e18
samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaYarnAppMasterService.scala
5f2bfc5
samza-yarn/src/main/scala/org/apache/samza/webapp/ApplicationMasterRestServlet.scala
cdd389c
Diff: https://reviews.apache.org/r/53163/diff/
Testing
-------
./gradlew clean build
./gradlew checkstyleMain checkstyleTest
Checked that app state is shown correctly on web servlet index page.
Thanks,
Branislav Cogic
