Re: Review Request 38390: Adding oversubscription summary.

2015-09-21 Thread Maxim Khutornenko
> 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

Re: Review Request 38390: Adding oversubscription summary.

2015-09-21 Thread Maxim Khutornenko
--- 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

Re: Review Request 38390: Adding oversubscription summary.

2015-09-21 Thread Maxim Khutornenko
> 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

Re: Review Request 38390: Adding oversubscription summary.

2015-09-21 Thread Zameer Manji
--- 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 38390: Adding oversubscription summary.

2015-09-15 Thread Maxim Khutornenko
> 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

Re: Review Request 38390: Adding oversubscription summary.

2015-09-15 Thread Maxim Khutornenko
> On Sept. 15, 2015, 8:59 p.m., Stephan Erb wrote: > > docs/configuration-reference.md, line 346 > > > > > > How about attaching a date or version classifier here? Makes it much > > easier for a reader to

Re: Review Request 38390: Adding oversubscription summary.

2015-09-15 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38390/ --- (Updated Sept. 16, 2015, 12:38 a.m.) Review request for Aurora and Bill

Re: Review Request 38390: Adding oversubscription summary.

2015-09-15 Thread Bill Farner
> On Sept. 14, 2015, 6:20 p.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

Re: Review Request 38390: Adding oversubscription summary.

2015-09-15 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38390/#review99137 --- Ship it! Master (44e4726) is green with this patch.

Re: Review Request 38390: Adding oversubscription summary.

2015-09-15 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38390/#review99006 --- Ship it! Master (44e4726) is green with this patch.

Re: Review Request 38390: Adding oversubscription summary.

2015-09-15 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38390/#review99091 --- docs/configuration-reference.md (line 346)

Re: Review Request 38390: Adding oversubscription summary.

2015-09-14 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38390/ --- (Updated Sept. 15, 2015, 12:54 a.m.) Review request for Aurora and Bill

Re: Review Request 38390: Adding oversubscription summary.

2015-09-14 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38390/#review98977 --- docs/configuration-reference.md (line 332)