> 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?
Good catch. I removed them. - Zameer ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35812/#review89391 ----------------------------------------------------------- On June 25, 2015, 5:07 p.m., Zameer Manji wrote: > > ----------------------------------------------------------- > 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 and Bill Farner. > > > Bugs: AURORA-1074 > https://issues.apache.org/jira/browse/AURORA-1074 > > > Repository: aurora > > > Description > ------- > > Remove the "enable_legacy_constraints" flag and associated behaviour. > > > Diffs > ----- > > NEWS 1a0fb48c3bed1761937e98fa528fac39f2bdc05b > > src/main/java/org/apache/aurora/scheduler/configuration/ConfigurationManager.java > b777777b0ebbf303778e528b16ff3db1aa4e76f1 > > src/test/java/org/apache/aurora/scheduler/app/local/simulator/ClusterSimulatorModule.java > 9ee4fe2e76d36e17f8de2ab3eb714a6aae52c09c > > src/test/java/org/apache/aurora/scheduler/configuration/ConfigurationManagerTest.java > abbd23dd3ee4382565ce846eb035e2aa502badae > src/test/java/org/apache/aurora/scheduler/mesos/Offers.java > 83eec5d3c8e493fcefbcb5b9cf67dae4e741b095 > > src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java > 71b09b1fddc5faa7960ade43929cb57eec3243dd > > Diff: https://reviews.apache.org/r/35812/diff/ > > > Testing > ------- > > ./gradlew build -Pq > > > Thanks, > > Zameer Manji > >