You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@samza.apache.org by Zhijie Shen <zs...@hortonworks.com> on 2014/04/02 23:16:41 UTC

Review Request 19963: SAMZA-58 Use YARN's AMRMClientAsync client library

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

Review request for samza.


Repository: samza


Description
-------

SAMZA-58 Use YARN's AMRMClientAsync client library

See comments: https://issues.apache.org/jira/browse/SAMZA-58?focusedCommentId=13958171&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13958171


Diffs
-----

  build.gradle 4775087 
  samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMaster.scala de6887d 
  samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMasterLifecycle.scala 5d09265 
  samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMasterTaskManager.scala 9058210 
  samza-yarn/src/main/scala/org/apache/samza/job/yarn/YarnAppMaster.scala e45c177 
  samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestSamzaAppMaster.scala PRE-CREATION 
  samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestSamzaAppMasterLifecycle.scala 4ee77e8 
  samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestSamzaAppMasterTaskManager.scala 9d832ae 
  samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestYarnAppMaster.scala 98f7844 

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


Testing
-------


Thanks,

Zhijie Shen


Re: Review Request 19963: SAMZA-58 Use YARN's AMRMClientAsync client library

Posted by Zhijie Shen <zs...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19963/
-----------------------------------------------------------

(Updated April 7, 2014, 9:03 a.m.)


Review request for samza.


Changes
-------

Update the patch according to Chris' comments


Repository: samza


Description
-------

SAMZA-58 Use YARN's AMRMClientAsync client library

See comments: https://issues.apache.org/jira/browse/SAMZA-58?focusedCommentId=13958171&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13958171


Diffs (updated)
-----

  build.gradle 75b6c98 
  samza-yarn/src/main/scala/org/apache/samza/config/YarnConfig.scala 6c3aa92 
  samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMaster.scala de6887d 
  samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMasterLifecycle.scala 5d09265 
  samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMasterMetrics.scala 983771d 
  samza-yarn/src/main/scala/org/apache/samza/job/yarn/SamzaAppMasterTaskManager.scala 9058210 
  samza-yarn/src/main/scala/org/apache/samza/job/yarn/YarnAppMaster.scala e45c177 
  samza-yarn/src/main/scala/org/apache/samza/job/yarn/YarnAppMasterListener.scala 1353e86 
  samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestSamzaAppMaster.scala PRE-CREATION 
  samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestSamzaAppMasterLifecycle.scala 4ee77e8 
  samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestSamzaAppMasterTaskManager.scala 9d832ae 
  samza-yarn/src/test/scala/org/apache/samza/job/yarn/TestYarnAppMaster.scala 98f7844 

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


Testing
-------


Thanks,

Zhijie Shen