Re: Review Request 43074: SAMZA-866 Refactor and fix Container allocation logic.

2016-02-02 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43074/#review117582 --- Ship it! lgtm! Please fix the nits pointed out by Jagadish. s

Re: Review Request 39032: SAMZA-787: task.log4j.system should not be guessed if not configured

2016-02-02 Thread Chinmay Soman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39032/#review117572 --- Ship it! - Chinmay Soman On Oct. 6, 2015, 12:44 a.m., Navina

Re: Review Request 43074: SAMZA-866 Refactor and fix Container allocation logic.

2016-02-02 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43074/#review117555 --- samza-yarn/src/main/java/org/apache/samza/job/yarn/AbstractContai

Re: Review Request 43053: allow warning instead of fail in case of invalid num of partitions in the checkpoint partition

2016-02-02 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43053/#review117554 --- samza-kafka/src/test/scala/org/apache/samza/checkpoint/kafka/Test

Re: Review Request 43053: allow warning instead of fail in case of invalid num of partitions in the checkpoint partition

2016-02-02 Thread Boris Shkolnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43053/ --- (Updated Feb. 3, 2016, 1:41 a.m.) Review request for samza. Changes ---