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

2016-11-03 Thread Shanthoosh Venkataraman
> On Nov. 3, 2016, 6:32 a.m., Prateek Maheshwari wrote: > > samza-core/src/main/scala/org/apache/samza/container/SamzaContainer.scala, > > line 104 > > > > > > Don't think we need a Util method for a string concat.

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

2016-11-03 Thread Shanthoosh Venkataraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52168/ --- (Updated Nov. 4, 2016, 1:04 a.m.) Review request for samza. Repository:

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

2016-11-03 Thread Boris Shkolnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53453/ --- 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-03 Thread Shanthoosh Venkataraman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52168/ --- (Updated Nov. 3, 2016, 11:19 p.m.) Review request for samza. Repository:

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

2016-11-03 Thread Prateek Maheshwari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52168/#review154675 --- docs/learn/documentation/versioned/rest/resources/tasks.md (line