Re: Review Request 30207: Simplify AuroraCommandContext

2015-01-28 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30207/#review70043 --- Bill, can I get a ship it here? - Zameer Manji On Jan. 22, 2015,

Re: Review Request 30207: Simplify AuroraCommandContext

2015-01-28 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30207/#review70046 --- Ship it! Ship It! - Bill Farner On Jan. 23, 2015, 3:32 a.m.,

Re: Review Request 30207: Simplify AuroraCommandContext

2015-01-27 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30207/#review69851 --- @ReviewBot retry - Bill Farner On Jan. 23, 2015, 3:32 a.m.,

Re: Review Request 30207: Simplify AuroraCommandContext

2015-01-27 Thread Maxim Khutornenko
On Jan. 23, 2015, 8:47 p.m., Maxim Khutornenko wrote: I am not convinced there is enough value in this diff to risk possible regression. Besides, the majority of what this diff touches will die out along with the client updater. Zameer Manji wrote: Is there an ETA for the

Re: Review Request 30207: Simplify AuroraCommandContext

2015-01-27 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30207/#review69824 --- Ship it! Ship It! - Maxim Khutornenko On Jan. 23, 2015, 3:32

Re: Review Request 30207: Simplify AuroraCommandContext

2015-01-26 Thread Bill Farner
On Jan. 23, 2015, 8:47 p.m., Maxim Khutornenko wrote: I am not convinced there is enough value in this diff to risk possible regression. Besides, the majority of what this diff touches will die out along with the client updater. Zameer Manji wrote: Is there an ETA for the

Re: Review Request 30207: Simplify AuroraCommandContext

2015-01-23 Thread Zameer Manji
On Jan. 23, 2015, 12:47 p.m., Maxim Khutornenko wrote: I am not convinced there is enough value in this diff to risk possible regression. Besides, the majority of what this diff touches will die out along with the client updater. Zameer Manji wrote: Is there an ETA for the

Re: Review Request 30207: Simplify AuroraCommandContext

2015-01-23 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30207/#review69458 --- I am not convinced there is enough value in this diff to risk

Re: Review Request 30207: Simplify AuroraCommandContext

2015-01-23 Thread Maxim Khutornenko
On Jan. 23, 2015, 8:47 p.m., Maxim Khutornenko wrote: I am not convinced there is enough value in this diff to risk possible regression. Besides, the majority of what this diff touches will die out along with the client updater. Zameer Manji wrote: Is there an ETA for the

Re: Review Request 30207: Simplify AuroraCommandContext

2015-01-23 Thread Zameer Manji
On Jan. 23, 2015, 12:47 p.m., Maxim Khutornenko wrote: I am not convinced there is enough value in this diff to risk possible regression. Besides, the majority of what this diff touches will die out along with the client updater. Is there an ETA for the destruction of the client

Re: Review Request 30207: Simplify AuroraCommandContext

2015-01-23 Thread Maxim Khutornenko
On Jan. 23, 2015, 8:47 p.m., Maxim Khutornenko wrote: I am not convinced there is enough value in this diff to risk possible regression. Besides, the majority of what this diff touches will die out along with the client updater. Zameer Manji wrote: Is there an ETA for the

Re: Review Request 30207: Simplify AuroraCommandContext

2015-01-23 Thread Maxim Khutornenko
On Jan. 23, 2015, 8:47 p.m., Maxim Khutornenko wrote: I am not convinced there is enough value in this diff to risk possible regression. Besides, the majority of what this diff touches will die out along with the client updater. Zameer Manji wrote: Is there an ETA for the

Re: Review Request 30207: Simplify AuroraCommandContext

2015-01-23 Thread Bill Farner
On Jan. 23, 2015, 8:47 p.m., Maxim Khutornenko wrote: I am not convinced there is enough value in this diff to risk possible regression. Besides, the majority of what this diff touches will die out along with the client updater. Zameer Manji wrote: Is there an ETA for the

Review Request 30207: Simplify AuroraCommandContext

2015-01-22 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30207/ --- Review request for Aurora, Maxim Khutornenko and Bill Farner. Repository: