Re: Review Request 35812: Remove "enable_legacy_constraints" flag.

2015-06-25 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35812/#review89453 --- Ship it! Ship It! - Maxim Khutornenko On June 26, 2015, 12:07 a.

Re: Review Request 35812: Remove "enable_legacy_constraints" flag.

2015-06-25 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35812/#review89452 --- Ship it! Master (56bb1e6) is green with this patch. ./build-suppo

Re: Review Request 35812: Remove "enable_legacy_constraints" flag.

2015-06-25 Thread Zameer Manji
> On June 25, 2015, 9:38 a.m., Bill Farner wrote: > > NEWS, line 5 > > > > > > ``` > > The scheduler command line argument enable_legacy_constraints has been > > removed, and the scheduler no longer automatically in

Re: Review Request 35812: Remove "enable_legacy_constraints" flag.

2015-06-25 Thread Zameer Manji
> On June 25, 2015, 10:40 a.m., Maxim Khutornenko wrote: > > src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java, > > lines 75-76 > > > > > > AFAICT, these are now only referenced in test

Re: Review Request 35812: Remove "enable_legacy_constraints" flag.

2015-06-25 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35812/ --- (Updated June 25, 2015, 5:07 p.m.) Review request for Aurora, Maxim Khutornenko

Re: Review Request 35812: Remove "enable_legacy_constraints" flag.

2015-06-25 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35812/#review89391 --- src/main/java/org/apache/aurora/scheduler/configuration/Configurati

Re: Review Request 35812: Remove "enable_legacy_constraints" flag.

2015-06-25 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35812/#review89373 --- Ship it! NEWS (line 5)

Re: Review Request 35812: Remove "enable_legacy_constraints" flag.

2015-06-23 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35812/#review89102 --- Ship it! Master (4b8c34c) is green with this patch. ./build-suppo

Review Request 35812: Remove "enable_legacy_constraints" flag.

2015-06-23 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35812/ --- Review request for Aurora, Maxim Khutornenko and Bill Farner. Bugs: AURORA-1074