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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43053/#review119748 --- Ship it! Just one nit. docs/learn/documentation/versioned/job

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43053/ --- (Updated Feb. 18, 2016, 8:04 p.m.) Review request for samza. Changes ---

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43053/ --- (Updated Feb. 18, 2016, 7:21 p.m.) Review request for samza. Changes ---

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

> On Feb. 10, 2016, 6:56 a.m., Yi Pan (Data Infrastructure) wrote: > > samza-core/src/main/scala/org/apache/samza/config/JobConfig.scala, line 46 > > > > > > One more nit that I forgot: it would be nice to document wh

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43053/ --- (Updated Feb. 16, 2016, 9:41 p.m.) Review request for samza. Bugs: SAMZA-864

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43053/#review118602 --- samza-core/src/main/scala/org/apache/samza/config/JobConfig.scala

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43053/#review118598 --- Ship it! Ship It! - Yi Pan (Data Infrastructure) On Feb. 5,

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43053/ --- (Updated Feb. 5, 2016, 9:48 p.m.) Review request for samza. Bugs: SAMZA-864

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43053/ --- (Updated Feb. 5, 2016, 9:46 p.m.) Review request for samza. Changes ---

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

> On Feb. 3, 2016, 2:36 a.m., Jagadish Venkatraman wrote: > > samza-kafka/src/test/scala/org/apache/samza/checkpoint/kafka/TestKafkaCheckpointManager.scala, > > line 84 > > > > > > nit: failOnTopicValidation. tnx, f

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

--- 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

--- 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 ---

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43053/#review117242 --- Fix it, then Ship it! LGTM. One minor comment below. samza-ka