Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45222/#review126219 --- Master (55a2422) is red with this patch. ./build-support/jenkins

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45222/ --- (Updated March 30, 2016, 4 p.m.) Review request for Aurora, Joshua Cohen and Ma

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

> On March 30, 2016, 8:47 p.m., Joshua Cohen wrote: > > Ship It! Can you rebase and fix the conflicts in RELEASE-NOTES.md? Then I'll commit this. - Joshua --- This is an automatically generated e-mail. To reply, visit: https://reviews.a

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45222/#review126196 --- Ship it! Ship It! - Joshua Cohen On March 29, 2016, 5:51 p.m

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

> On March 30, 2016, 9:49 a.m., Joshua Cohen wrote: > > Code-wise this looks fine to me, however, I have some reservations about > > making tier required. I think that throwing a `TaskDescriptionError` when > > tier is defined, but not valid is fine, but can we/should we continue to > > defaul

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

> On March 30, 2016, 9:49 a.m., Joshua Cohen wrote: > > Code-wise this looks fine to me, however, I have some reservations about > > making tier required. I think that throwing a `TaskDescriptionError` when > > tier is defined, but not valid is fine, but can we/should we continue to > > defaul

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

> On March 30, 2016, 9:49 a.m., Joshua Cohen wrote: > > Code-wise this looks fine to me, however, I have some reservations about > > making tier required. I think that throwing a `TaskDescriptionError` when > > tier is defined, but not valid is fine, but can we/should we continue to > > defaul

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45222/#review126132 --- Code-wise this looks fine to me, however, I have some reservations

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45222/#review125922 --- Ship it! Master (ec29ac1) is green with this patch. ./build-s

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45222/ --- (Updated March 29, 2016, 10:51 a.m.) Review request for Aurora, Joshua Cohen an

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

> On March 28, 2016, 1:23 p.m., Bill Farner wrote: > > I won't have time to review this, so i'd like to stand down. > > > > However, please make sure to add appropriate release notes, specifically > > for the stated backwards incompatibility. Will do. I will post an updated review request with

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45222/#review125737 --- I won't have time to review this, so i'd like to stand down. Howe

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45222/#review125729 --- This patch does not apply cleanly against master (0950095), do you

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45222/ --- (Updated March 28, 2016, 12:43 p.m.) Review request for Aurora, Joshua Cohen, M

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

> On March 28, 2016, 9:36 a.m., Maxim Khutornenko wrote: > > src/test/java/org/apache/aurora/scheduler/preemptor/PreemptionVictimFilterTest.java, > > line 131 > > > > > > Any reason to allow more than "once" (defaul

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

> On March 28, 2016, 9:36 a.m., Maxim Khutornenko wrote: > > src/test/java/org/apache/aurora/scheduler/preemptor/PreemptionVictimFilterTest.java, > > line 131 > > > > > > Any reason to allow more than "once" (defaul

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45222/#review125671 --- Ship it! src/test/java/org/apache/aurora/scheduler/preemptor/P

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45222/#review125496 --- Ship it! Master (b352e8b) is green with this patch. ./build-s

Re: Review Request 45222: Introduce "preemptible" flag in TierInfo with backward compatible support for "production" flag in TaskConfig.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45222/ --- (Updated March 25, 2016, 5:08 p.m.) Review request for Aurora, Joshua Cohen, Ma