Re: Review Request 27591: Remove --bind-var for cron deschedule.

2014-11-05 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27591/#review60005 --- Ship it! Thanks! This is now on master $ git log -1

Review Request 27591: Remove --bind-var for cron deschedule.

2014-11-04 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27591/ --- Review request for Aurora and Bill Farner. Bugs: AURORA-854

Re: Review Request 27591: Remove --bind-var for cron deschedule.

2014-11-04 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27591/#review59851 --- Master (9847b7b) is red with this patch.

Re: Review Request 27591: Remove --bind-var for cron deschedule.

2014-11-04 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27591/#review59869 --- Can you re-post your diff to prove a green response from the build

Re: Review Request 27591: Remove --bind-var for cron deschedule.

2014-11-04 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27591/ --- (Updated Nov. 4, 2014, 6:52 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 27591: Remove --bind-var for cron deschedule.

2014-11-04 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27591/#review59923 --- Master (9847b7b) is red with this patch.

Re: Review Request 27591: Remove --bind-var for cron deschedule.

2014-11-04 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27591/#review59924 --- Bill, what should I do now? - Zameer Manji On Nov. 4, 2014, 6:52

Re: Review Request 27591: Remove --bind-var for cron deschedule.

2014-11-04 Thread Bill Farner
On Nov. 5, 2014, 3:12 a.m., Zameer Manji wrote: Bill, what should I do now? I'll run it locally to confirm the patch. - Bill --- This is an automatically generated e-mail. To reply, visit: