Re: Review Request 53453: Add optional interface for SystemConsumer checkpontListener() for checkpoint notifications

2016-11-04 Thread Boris Shkolnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53453/ --- (Updated Nov. 4, 2016, 11:23 p.m.) Review request for samza. Bugs: SAMZA-1042

Re: Review Request 52168: Tasks endpoint to list the complete details of all tasks related to a job

2016-11-04 Thread Jagadish Venkatraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52168/#review154972 --- Draft review so that I don't loose comments. Overall, it's hard t

Re: Review Request 52168: Tasks endpoint to list the complete details of all tasks related to a job

2016-11-04 Thread Prateek Maheshwari
> On Nov. 2, 2016, 11:32 p.m., Prateek Maheshwari wrote: > > docs/learn/documentation/versioned/rest/resources/tasks.md, line 55 > > > > > > What's the difference b/w containerId and containerName? > > Shanthoosh Ven

Re: Review Request 52168: Tasks endpoint to list the complete details of all tasks related to a job

2016-11-04 Thread Prateek Maheshwari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52168/#review154832 --- samza-core/src/main/scala/org/apache/samza/util/Util.scala (line