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

(Updated June 13, 2016, 11:52 p.m.)


Review request for samza and Yi Pan (Data Infrastructure).


Changes
-------

addressed Yis comments and rebased with current master.


Repository: samza


Description
-------

Integrate Kerberos with the refactored JC patch as in RB:47283.


Diffs (updated)
-----

  
samza-core/src/main/java/org/apache/samza/clustermanager/AbstractContainerAllocator.java
 097a476bee75bb2b5a56ae85317fa16523baebd3 
  
samza-core/src/main/java/org/apache/samza/clustermanager/ContainerProcessManager.java
 f2db34c24b2b2ef05705a2fbf00d4a211d6f4c58 
  
samza-core/src/main/java/org/apache/samza/clustermanager/SamzaApplicationState.java
 ca277b3636aa4e98e639699e657de6aced613c0f 
  
samza-core/src/main/scala/org/apache/samza/metrics/ContainerProcessManagerMetrics.scala
 86c2440ad1d253fd71030916915984c1d08614fb 
  
samza-core/src/test/java/org/apache/samza/clustermanager/TestContainerAllocator.java
 f1475709c2db364110b00cb04cf55c6ecb44fd04 
  
samza-core/src/test/java/org/apache/samza/clustermanager/TestContainerProcessManager.java
 4fd1018f93dcfbccf70595c4bcdec956d2b59295 
  
samza-core/src/test/java/org/apache/samza/clustermanager/TestHostAwareContainerAllocator.java
 57fef126de535067a68ce6eb8eeddf3334d1fe5d 
  samza-shell/src/main/bash/run-am.sh ca938cc910b8b59e2cdab68df9f9ee16b425fb74 
  
samza-yarn/src/main/java/org/apache/samza/job/yarn/AbstractContainerAllocator.java
 b4789e62beb1120f11a8101664b10c34ae930e58 
  samza-yarn/src/main/java/org/apache/samza/job/yarn/ContainerAllocator.java 
24ac41067e34cf5a445bb036db9ea324eaafa7df 
  samza-yarn/src/main/java/org/apache/samza/job/yarn/ContainerFailure.java 
1d15651096e52a2e323a8e2f658fad3ea8e9c709 
  samza-yarn/src/main/java/org/apache/samza/job/yarn/ContainerRequestState.java 
57ce35099973b7fc3414c450e3246cb9f204289b 
  samza-yarn/src/main/java/org/apache/samza/job/yarn/ContainerUtil.java 
e8976bce934a324c48475bcd64d392119cc44b40 
  
samza-yarn/src/main/java/org/apache/samza/job/yarn/HostAwareContainerAllocator.java
 1d101fa80e6367d54f06455c242222195e4c0091 
  samza-yarn/src/main/java/org/apache/samza/job/yarn/SamzaAppState.java 
c116ed83f8b424bc7353e89c60cf165da542fbec 
  samza-yarn/src/main/java/org/apache/samza/job/yarn/SamzaContainerRequest.java 
4a04eb6b3b054ee85988e1b26ececc800bbc7861 
  samza-yarn/src/main/java/org/apache/samza/job/yarn/SamzaTaskManager.java 
bc95f31c0dcaaa68d483a6f152b61aba6c543fff 
  samza-yarn/src/main/java/org/apache/samza/job/yarn/YarnAppState.java 
57092e1b100b0128aed2a1426835d7184cc5bef7 
  
samza-yarn/src/main/java/org/apache/samza/job/yarn/YarnClusterResourceManager.java
 7778a380dcd99139012a71c80fb86ede1fef0f71 
  samza-yarn/src/main/resources/scalate/WEB-INF/views/index.scaml 
93660c73e507b1a6076e743fcec56a41d22a5f39 
  samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMaster.scala 
1fb18bed9e6dfbbc45a1e277279d9513757d17bd 
  
samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMasterLifecycle.scala
 2a5c0d8092aa6f0a7f9b1f5fa56f9f4d4919d579 
  
samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMasterMetrics.scala 
054d8b68033535ff0cab7cca84b71455e201a715 
  
samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMasterService.scala 
979d81d1b9871e0a1aa9e3c0fec7c13464392474 
  
samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaYarnAppMasterLifecycle.scala
 2ed9baffc483d180aa6f5ff9e32abf9d1e1aef6f 
  
samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaYarnAppMasterService.scala
 f62bec1b305b7301829644b5558eac197ea7457e 
  samza-yarn/src/main/scala/org/apache/samza/job/yarn/YarnJob.scala 
3ca1f0db59127d027604b015ddba07a550ceb114 
  
samza-yarn/src/main/scala/org/apache/samza/webapp/ApplicationMasterRestServlet.scala
 a40ab72a71c4fa82bd87aa03caec8936e609bf68 
  
samza-yarn/src/main/scala/org/apache/samza/webapp/ApplicationMasterWebServlet.scala
 605332a02f365248fc9ec3525ed9d3721db71e8d 
  
samza-yarn/src/test/java/org/apache/samza/job/yarn/TestContainerAllocator.java 
e21aded3c5e302593ed1cb5675da81aa6e3e743f 
  
samza-yarn/src/test/java/org/apache/samza/job/yarn/TestContainerAllocatorCommon.java
 0bbd48d8720a79f3b453d69ca2772f0db10723be 
  
samza-yarn/src/test/java/org/apache/samza/job/yarn/TestContainerRequestState.java
 402fe784120ac40ed542d9fa60d6a6d7df9c8cda 
  
samza-yarn/src/test/java/org/apache/samza/job/yarn/TestHostAwareContainerAllocator.java
 ead7200487f27ed31e30576721ef16a064a28bc7 
  
samza-yarn/src/test/java/org/apache/samza/job/yarn/TestSamzaContainerRequest.java
 ad0f4d3d9db9111a7c0087b6da9c724dcc736726 
  samza-yarn/src/test/java/org/apache/samza/job/yarn/TestSamzaTaskManager.java 
d747b818ffdf0c9968afdf098b2a3f14b7b02a56 
  
samza-yarn/src/test/java/org/apache/samza/job/yarn/util/MockContainerAllocator.java
 329024729eeae07f8e755777b7211772cd98e04e 
  
samza-yarn/src/test/java/org/apache/samza/job/yarn/util/MockContainerRequestState.java
 7c0b50437b06230ab840fdad894516c6d8545b4a 
  
samza-yarn/src/test/java/org/apache/samza/job/yarn/util/MockContainerUtil.java 
cf3e1439d005ca03f8c0e84183a26a0ca85abd0a 
  samza-yarn/src/test/java/org/apache/samza/job/yarn/util/TestUtil.java 
d4c9c96144c58e1acc53726265a452f2467cd554 
  samza-yarn/src/test/scala/org/apache/samza/job/yarn/MockSystemAdmin.scala 
PRE-CREATION 
  samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestSamzaAppMaster.scala 
3f056c4d39fc8e3a2ed42fbcc73aa6eca1a2287c 
  
samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestSamzaAppMasterLifecycle.scala
 750f467da12c5eb7ba80b06faadb1bac12feac1b 
  
samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestSamzaAppMasterService.scala
 3de5614c56de9d0427fcfa22a864ab2df47aa095 

Diff: https://reviews.apache.org/r/48393/diff/


Testing
-------

Manual testing.

Added Unit tests.

Manually tested with a sample hello-world job.

Tested that the UI was showing up correctly and template files were being 
populated. f


Thanks,

Jagadish Venkatraman

Reply via email to