Re: Review Request 45933: Fix bug when checking to ensure that all db migrations on the classpath have been applied.

2016-04-08 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45933/#review127822 --- Ship it! Ship It! - Maxim Khutornenko On April 8, 2016,

Re: Review Request 45935: Remove support for canonical command line arg names.

2016-04-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45935/ --- (Updated April 8, 2016, 11:26 a.m.) Review request for Aurora and Zameer

Re: Review Request 45902: Introduce a Curator-based `ServiceGroupMonitor`.

2016-04-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45902/#review127817 ---

Re: Review Request 45933: Fix bug when checking to ensure that all db migrations on the classpath have been applied.

2016-04-08 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45933/#review127832 --- @ReviewBot retry - Joshua Cohen On April 8, 2016, 5:44 p.m.,

Re: Review Request 45936: Remove support for reading command line argument values from files.

2016-04-08 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45936/#review127837 --- Ship it! Ship It! - Zameer Manji On April 8, 2016, 11:32

Re: Review Request 45935: Remove support for canonical command line arg names.

2016-04-08 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45935/#review127834 --- Master (5ecded7) is red with this patch.

Review Request 45936: Remove support for reading command line argument values from files.

2016-04-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45936/ --- Review request for Aurora and Zameer Manji. Repository: aurora Description

Review Request 45935: Remove support for canonical command line arg names.

2016-04-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45935/ --- Review request for Aurora and Zameer Manji. Repository: aurora Description

Re: Review Request 45935: Remove support for canonical command line arg names.

2016-04-08 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45935/#review127831 --- This patch does not apply cleanly against master (5ecded7), do

Re: Review Request 45933: Fix bug when checking to ensure that all db migrations on the classpath have been applied.

2016-04-08 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45933/#review127835 --- Master (5ecded7) is red with this patch.

Re: Review Request 45935: Remove support for canonical command line arg names.

2016-04-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45935/#review127839 --- @ReviewBot retry - Bill Farner On April 8, 2016, 11:26 a.m.,

Re: Review Request 45933: Fix bug when checking to ensure that all db migrations on the classpath have been applied.

2016-04-08 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45933/#review127840 --- @ReviewBot retry - Joshua Cohen On April 8, 2016, 5:44 p.m.,

Re: Review Request 45933: Fix bug when checking to ensure that all db migrations on the classpath have been applied.

2016-04-08 Thread Joshua Cohen
> On April 8, 2016, 6:39 p.m., Joshua Cohen wrote: > > @ReviewBot retry Looks like archive.apache.org is down for maintenance until Monday. Shipping this sans-ReviewBot's thumbs up. - Joshua --- This is an automatically generated

Review Request 45933: Fix bug when checking to ensure that all db migrations on the classpath have been applied.

2016-04-08 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45933/ --- Review request for Aurora and Maxim Khutornenko. Repository: aurora

Re: Review Request 45939: Remove support for positional command line arguments,

2016-04-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45939/ --- (Updated April 8, 2016, 12:27 p.m.) Review request for Aurora and Zameer

Review Request 45939: Remove support for positional command line arguments,

2016-04-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45939/ --- Review request for Aurora and Zameer Manji. Repository: aurora Description

Re: Review Request 45936: Remove support for reading command line argument values from files.

2016-04-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45936/ --- (Updated April 8, 2016, 12:20 p.m.) Review request for Aurora and Zameer