Re: Review Request 50082: SAMZA-973: Disk Quotas: clamp max delay, better measure processing time

2016-07-18 Thread Chris Pettitt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50082/ --- (Updated July 18, 2016, 3:29 p.m.) Review request for samza. Repository: samz

Re: Review Request 50082: SAMZA-973: Disk Quotas: clamp max delay, better measure processing time

2016-07-18 Thread Chris Pettitt
> On July 15, 2016, 10:52 p.m., Xinyu Liu wrote: > > samza-core/src/main/scala/org/apache/samza/container/RunLoop.scala, line 80 > > > > > > Shall we move this line in the process() function? I'll move it into proces

Re: Review Request 50082: SAMZA-973: Disk Quotas: clamp max delay, better measure processing time

2016-07-15 Thread Xinyu Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50082/#review142459 --- Fix it, then Ship it! samza-core/src/main/scala/org/apache/sam

Re: Review Request 50082: SAMZA-973: Disk Quotas: clamp max delay, better measure processing time

2016-07-15 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50082/#review142457 --- Ship it! +1. lgtm. - Yi Pan (Data Infrastructure) On July 15

Re: Review Request 50082: SAMZA-973: Disk Quotas: clamp max delay, better measure processing time

2016-07-15 Thread Chris Pettitt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50082/ --- (Updated July 15, 2016, 7:59 p.m.) Review request for samza. Repository: samz

Review Request 50082: SAMZA-973: Disk Quotas: clamp max delay, better measure processing time

2016-07-15 Thread Chris Pettitt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50082/ --- Review request for samza. Repository: samza Description --- The delay ti