Re: Review Request 41226: Handling task event race in updater.

2015-12-16 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41226/#review110696 --- Ship it! Master (fb8155d) is green with this patch. true I

Re: Review Request 41226: Handling task event race in updater.

2015-12-15 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41226/#review110614 --- Ping. - Maxim Khutornenko On Dec. 11, 2015, 1:33 a.m., Maxim

Re: Review Request 41226: Handling task event race in updater.

2015-12-11 Thread Maxim Khutornenko
> On Dec. 11, 2015, 7:07 p.m., Bill Farner wrote: > > Can you take a stab at illustrating in the description how this problem > > manifests? The current shape of the patch feels like it papers over a > > problem that we might be left to solve again, so thinking about the problem > > at a

Re: Review Request 41226: Handling task event race in updater.

2015-12-11 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41226/#review109984 --- Can you take a stab at illustrating in the description how this

Re: Review Request 41226: Handling task event race in updater.

2015-12-10 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41226/ --- (Updated Dec. 11, 2015, 1:33 a.m.) Review request for Aurora and Bill Farner.

Re: Review Request 41226: Handling task event race in updater.

2015-12-10 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41226/#review109906 --- Ship it! Master (6933a71) is green with this patch.