Re: Review Request 19888: Prepare to store UNKNOWN state.

2014-04-03 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19888/ --- (Updated April 3, 2014, 5:31 p.m.) Review request for Aurora, Kevin Sweeney

Re: Review Request 19888: Prepare to store UNKNOWN state.

2014-04-01 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19888/#review39207 --- Isn't a StorageBackfill what you're after here? Rewrite tasks in

Re: Review Request 19888: Prepare to store UNKNOWN state.

2014-04-01 Thread Maxim Khutornenko
On April 1, 2014, 10:43 p.m., Bill Farner wrote: Isn't a StorageBackfill what you're after here? Rewrite tasks in the UNKNOWN state to a more appropriate state? The StorageBackfill is a much more invasive approach, which is not really required here. Besides, rewriting state would mess

Re: Review Request 19888: Prepare to store UNKNOWN state.

2014-04-01 Thread Bill Farner
On April 1, 2014, 10:43 p.m., Bill Farner wrote: Isn't a StorageBackfill what you're after here? Rewrite tasks in the UNKNOWN state to a more appropriate state? Maxim Khutornenko wrote: The StorageBackfill is a much more invasive approach, which is not really required here.

Re: Review Request 19888: Prepare to store UNKNOWN state.

2014-04-01 Thread Maxim Khutornenko
On April 1, 2014, 10:43 p.m., Bill Farner wrote: Isn't a StorageBackfill what you're after here? Rewrite tasks in the UNKNOWN state to a more appropriate state? Maxim Khutornenko wrote: The StorageBackfill is a much more invasive approach, which is not really required here.

Re: Review Request 19888: Prepare to store UNKNOWN state.

2014-04-01 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19888/ --- (Updated April 2, 2014, 12:05 a.m.) Review request for Aurora, Kevin Sweeney

Re: Review Request 19888: Prepare to store UNKNOWN state.

2014-04-01 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19888/#review39220 --- Ship it!