Re: Review Request 53282: SAMZA-1043: Samza performance improvements

2016-11-02 Thread Xinyu Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53282/ --- (Updated Nov. 2, 2016, 5:56 p.m.) Review request for samza, Chris Pettitt, Jake

Re: Review Request 53282: SAMZA-1043: Samza performance improvements

2016-11-02 Thread Chris Pettitt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53282/#review154611 --- Fix it, then Ship it! Can you get the numbers with all fixes bu

Re: Review Request 53282: SAMZA-1043: Samza performance improvements

2016-11-02 Thread Prateek Maheshwari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53282/#review154602 --- samza-core/src/main/java/org/apache/samza/task/AsyncRunLoop.java

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

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

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

2016-11-02 Thread Shanthoosh Venkataraman
> On Oct. 25, 2016, 9:06 p.m., Navina Ramesh wrote: > > samza-core/src/main/scala/org/apache/samza/job/JobRunner.scala, line 47 > > > > > > I would think the ideal place for this method will be in Util as > > opposed

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

2016-11-02 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