Re: Review Request 38385: Documenting dedicated job & quota relationship.

> On Sept. 16, 2015, 12:51 a.m., Bill Farner wrote: > > docs/resource-isolation.md, line 150 > > > > > > This doc is otherewise about machine-level resource isolation, so it > > seems like an odd match here. It see

Re: Review Request 38385: Documenting dedicated job & quota relationship.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38385/ --- (Updated Sept. 21, 2015, 9:20 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 38390: Adding oversubscription summary.

> On Sept. 15, 2015, 1:20 a.m., Bill Farner wrote: > > docs/configuration-reference.md, lines 344-355 > > > > > > Tiers seems like a significant enough topic to warrant its own page > > with some more context and be

Re: Review Request 38390: Adding oversubscription summary.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38390/#review99835 --- Ship it! LGTM modulo my one nit. docs/configuration-reference.md

Re: Review Request 38385: Documenting dedicated job & quota relationship.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38385/#review99837 --- Ship it! LGTM modulo my nit below. docs/resource-isolation.md (li

Re: Review Request 38385: Documenting dedicated job & quota relationship.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38385/#review99842 --- Ship it! Master (1ff336e) is green with this patch. ./build-suppo

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

> On Sept. 16, 2015, 12:31 a.m., Bill Farner wrote: > > src/main/python/apache/aurora/client/api/command_runner.py, line 94 > > > > > > This patch uses mutable lists in several places, which i feel is a > > divergenc

Re: Review Request 38385: Documenting dedicated job & quota relationship.

> On Sept. 21, 2015, 10:12 p.m., Zameer Manji wrote: > > docs/resource-isolation.md, line 168 > > > > > > higher or equal priority. Negative: https://github.com/apache/aurora/blob/0070a5fd18c6f219a7fe66f327209b8dc2

Re: Review Request 38390: Adding oversubscription summary.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38390/ --- (Updated Sept. 21, 2015, 10:36 p.m.) Review request for Aurora, Bill Farner and

Re: Review Request 38390: Adding oversubscription summary.

> On Sept. 21, 2015, 9:48 p.m., Zameer Manji wrote: > > docs/configuration-reference.md, line 355 > > > > > > What happens if a job with a tier of revocable is sent to a scheduler > > where it isn't configured? I th

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37818/ --- (Updated Sept. 21, 2015, 11:42 p.m.) Review request for Aurora and Bill Farner.

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 2

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37818/#review99873 --- This patch does not apply cleanly on master (5627611), do you need t