Re: Review Request 42801: `TaskHistoryPruner` controls Lifecycle directly.

2016-01-26 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42801/#review116397 --- Ship it! Ship It! - Zameer Manji On Jan. 26, 2016, 7 a.m., J

Re: Review Request 42801: `TaskHistoryPruner` controls Lifecycle directly.

2016-01-26 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42801/#review116396 --- Ship it! Ship It! - Bill Farner On Jan. 26, 2016, 7 a.m., Jo

Re: Review Request 42801: `TaskHistoryPruner` controls Lifecycle directly.

2016-01-26 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42801/#review116361 --- Ship it! Master (39815a3) is green with this patch. ./build-s

Re: Review Request 42801: `TaskHistoryPruner` controls Lifecycle directly.

2016-01-26 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42801/ --- (Updated Jan. 26, 2016, 8 a.m.) Review request for Aurora, Bill Farner and Zame

Re: Review Request 42801: `TaskHistoryPruner` controls Lifecycle directly.

2016-01-26 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42801/#review116359 --- @ReviewBot retry - John Sirois On Jan. 26, 2016, 8 a.m., John S

Re: Review Request 42801: `TaskHistoryPruner` controls Lifecycle directly.

2016-01-26 Thread John Sirois
> On Jan. 26, 2016, 7:57 a.m., Aurora ReviewBot wrote: > > Can only test reviews with 1 parent, found 2: > > https://reviews.apache.org/api/review-requests/42639/ > > https://reviews.apache.org/api/review-requests/42332/ > > > > I will refresh this build result if you post a review conta

Re: Review Request 42801: `TaskHistoryPruner` controls Lifecycle directly.

2016-01-26 Thread John Sirois
- John --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42801/#review116356 --- On Jan. 26, 2016, 8 a.m., John Sirois wrote: > > ---

Re: Review Request 42801: `TaskHistoryPruner` controls Lifecycle directly.

2016-01-26 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42801/#review116356 --- Can only test reviews with 1 parent, found 2: https://revi

Review Request 42801: `TaskHistoryPruner` controls Lifecycle directly.

2016-01-26 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42801/ --- Review request for Aurora, Bill Farner and Zameer Manji. Bugs: AURORA-1593