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-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 https://reviews.apache.org/r/35812/diff/1/?file=990888#file990888line75 AFAICT, these are now only referenced in test now. Remove?

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

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 https://reviews.apache.org/r/35812/diff/1/?file=990887#file990887line5 ``` The scheduler command line argument enable_legacy_constraints has been removed, and the scheduler no longer automatically injects 'host'

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

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: