You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@samza.apache.org by Tao Feng <fe...@gmail.com> on 2016/04/15 05:55:03 UTC

Re: Review Request 41106: SAMZA-833: ProcessJob mishandling containers

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

(Updated April 15, 2016, 3:55 a.m.)


Review request for samza.


Repository: samza


Description (updated)
-------

rebase branch


Diffs (updated)
-----

  samza-core/src/main/scala/org/apache/samza/job/local/ProcessJobFactory.scala 17c2e5be9ee216c88e3c07784c4f9c05cd92e9c9 

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


Testing
-------

1. ./gradlew clean build
2. ./gradlew checkstyleMain checkstyleTest
3. manual test open source hello samza and verify that by giving yarn.container.count or job.container.count with value greater than 1 for ProcessJobFactory, the test will throw the desired exception.


Thanks,

Tao Feng