Re: Review Request 37818: Moved executor settings configuration to loadable JSON

> On Sept. 2, 2015, 4:45 p.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/configuration/ExecutorSettingsLoader.java, > > line 87 > > > > > > This code will be easier to maintain long-term with

Re: Review Request 37818: Moved executor settings configuration to loadable JSON

> On Sept. 2, 2015, 4:45 p.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/configuration/ExecutorSettingsLoader.java, > > line 87 > > > > > > This code will be easier to maintain long-term with

Re: Review Request 37818: Moved executor settings configuration to loadable JSON

> On Sept. 2, 2015, 9:45 a.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/configuration/ExecutorSettingsLoader.java, > > line 87 > > > > > > This code will be easier to maintain long-term with

Re: Review Request 37818: Moved executor settings configuration to loadable JSON

> On Sept. 2, 2015, 9:45 a.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/configuration/ExecutorSettingsLoader.java, > > line 87 > > > > > > This code will be easier to maintain long-term with

Re: Review Request 38280: Restore build properties within Scheduler vars endpoint and snapshots

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38280/ --- (Updated Sept. 16, 2015, 2:53 p.m.) Review request for Aurora, Maxim

Re: Review Request 38280: Restore build properties within Scheduler vars endpoint and snapshots

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38280/#review99317 --- Ship it! Master (9dff057) is green with this patch.

Re: Review Request 38280: Restore build properties within Scheduler vars endpoint and snapshots

> On Sept. 15, 2015, 4:25 p.m., Maxim Khutornenko wrote: > > src/test/java/org/apache/aurora/scheduler/storage/backup/RecoveryTest.java, > > line 76 > > > > > > Revert Done - Joe

Re: Review Request 38280: Restore build properties within Scheduler vars endpoint and snapshots

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38280/#review99312 --- I'm proposing some changes in this review which I think will

Re: Review Request 38280: Restore build properties within Scheduler vars endpoint and snapshots

> On Sept. 16, 2015, 2:54 p.m., Zameer Manji wrote: > > I'm proposing some changes in this review which I think will greatly > > simplify the code. > > Maxim Khutornenko wrote: > I agree there are ways to improve this functionality but given this is > mostly a restore of the old (and

Re: Review Request 38280: Restore build properties within Scheduler vars endpoint and snapshots

> On Sept. 16, 2015, 9:54 p.m., Zameer Manji wrote: > > I'm proposing some changes in this review which I think will greatly > > simplify the code. I agree there are ways to improve this functionality but given this is mostly a restore of the old (and proven) commons code can we move these

Re: Review Request 38280: Restore build properties within Scheduler vars endpoint and snapshots

> On Sept. 16, 2015, 2:54 p.m., Zameer Manji wrote: > > I'm proposing some changes in this review which I think will greatly > > simplify the code. > > Maxim Khutornenko wrote: > I agree there are ways to improve this functionality but given this is > mostly a restore of the old (and

Re: Review Request 38326: Adding ssh options into "aurora task" commands.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38326/#review99252 --- Ship it! Ship It! - Zameer Manji On Sept. 11, 2015, 4:31 p.m.,

Re: Review Request 38280: Restore build properties within Scheduler vars endpoint and snapshots

> On Sept. 15, 2015, 5:32 p.m., Bill Farner wrote: > > This patch is looking good to me, but i won't have time to give it a fair > > look as i head out of town. I'd like to tap out so you don't have to wait > > 3 weeks to land it :-) Roger, thanks! - Joe

Re: Review Request 38280: Restore build properties within Scheduler vars endpoint and snapshots

I can replace Bill on this review. On Wed, Sep 16, 2015 at 10:30 AM, Joe Smith wrote: > > > > On Sept. 15, 2015, 5:32 p.m., Bill Farner wrote: > > > This patch is looking good to me, but i won't have time to give it a > fair look as i head out of town. I'd like to tap out