Re: Review Request 36692: SAMZA-717 Expose the TaskNameGrouper API

2015-07-27 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36692/#review93097 --- Ship it! The code LGTM. I just realized that when we expose this vi

Re: Review Request 36692: SAMZA-717 Expose the TaskNameGrouper API

2015-07-23 Thread József Márton Jung
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36692/ --- (Updated July 23, 2015, 11:23 a.m.) Review request for samza. Repository: sam

Re: Review Request 36692: SAMZA-717 Expose the TaskNameGrouper API

2015-07-23 Thread József Márton Jung
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36692/ --- (Updated July 23, 2015, 11:16 a.m.) Review request for samza. Changes ---

Re: Review Request 36692: SAMZA-717 Expose the TaskNameGrouper API

2015-07-22 Thread Gustavo Anatoly F . V . Solís
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36692/#review92594 --- samza-api/src/main/java/org/apache/samza/job/model/ContainerModel.j