Re: Review Request 43237: Add benchmark for feching tasks.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43237/#review117940 --- Master (505f310) is red with this patch. ./build-support/jenkins

Review Request 43237: Add benchmark for feching tasks.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43237/ --- Review request for Aurora, Maxim Khutornenko and Bill Farner. Repository: auror

Re: Review Request 43237: Add benchmark for feching tasks.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43237/ --- (Updated Feb. 4, 2016, 6:04 p.m.) Review request for Aurora, Maxim Khutornenko

Re: Review Request 43172: Add deprecated field storage backfill

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43172/#review117902 --- Cherry-picked reverted patches and ran upgrade/rollback cycle twic

Re: Review Request 43027: Optionally enable setuid inside Docker containers

> On Feb. 4, 2016, 6:36 a.m., John Sirois wrote: > > Adding explicit reviewers to the People field is the standard way of moving > > an RB forward. As soon as your ready to have this looked at again, please > > add reviewers and give the all clear. Will do, thank you. - Benjamin -

Re: Review Request 43172: Add deprecated field storage backfill

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

Re: Review Request 43172: Add deprecated field storage backfill

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43172/ --- (Updated Feb. 4, 2016, 8:58 p.m.) Review request for Aurora, John Sirois and Bi

Re: Review Request 43172: Add deprecated field storage backfill

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

Re: Review Request 43172: Add deprecated field storage backfill

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43172/#review117839 --- Ship it! Ship It! - John Sirois On Feb. 4, 2016, 10:13 a.m.,

Re: Review Request 43172: Add deprecated field storage backfill

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43172/ --- (Updated Feb. 4, 2016, 5:13 p.m.) Review request for Aurora, John Sirois and Bi

Re: Review Request 43172: Add deprecated field storage backfill

> On Feb. 4, 2016, 10:37 a.m., Stephan Erb wrote: > > During this whole odyssee, have we learned a thing which is not reflected > > here? > > https://github.com/apache/aurora/blob/master/docs/thrift-deprecation.md > > > > If yes, please write it down :-) > > John Sirois wrote: > I concur,

Re: Review Request 43172: Add deprecated field storage backfill

> On Feb. 4, 2016, 4:36 p.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/storage/log/ThriftBackfill.java, > > line 47 > > > > > > Please favor immutable inputs and outputs; return `IJobUpdate`.

Re: Review Request 43172: Add deprecated field storage backfill

> On Feb. 4, 2016, 2:21 p.m., John Sirois wrote: > > src/test/java/org/apache/aurora/scheduler/storage/log/ThriftBackfillTest.java, > > line 50 > > > > > > Finish this thought - there is no actual testing done here.

Re: Review Request 43172: Add deprecated field storage backfill

> On Feb. 4, 2016, 6:21 a.m., John Sirois wrote: > > src/main/java/org/apache/aurora/scheduler/storage/db/TaskConfigManager.java, > > line 45 > > > > > > Having a backfill in the db layer strikes me as a red flag; ie

Re: Review Request 43172: Add deprecated field storage backfill

> On Feb. 4, 2016, 2:21 p.m., John Sirois wrote: > > src/main/java/org/apache/aurora/scheduler/storage/db/TaskConfigManager.java, > > line 45 > > > > > > Having a backfill in the db layer strikes me as a red flag; ie

Re: Review Request 43172: Add deprecated field storage backfill

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43172/#review117831 --- Ship it! src/main/java/org/apache/aurora/scheduler/storage/log

Re: Review Request 43172: Add deprecated field storage backfill

> On Feb. 4, 2016, 6:21 a.m., John Sirois wrote: > > src/main/java/org/apache/aurora/scheduler/storage/db/TaskConfigManager.java, > > line 45 > > > > > > Having a backfill in the db layer strikes me as a red flag; ie

Re: Review Request 43178: Remove unused entry in TaskMapper.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43178/#review117824 --- Ship it! Ship It! - Maxim Khutornenko On Feb. 4, 2016, 7:04

Re: Review Request 43027: Optionally enable setuid inside Docker containers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43027/#review117814 --- Adding explicit reviewers to the People field is the standard way

Re: Review Request 42984: Add failed result email protocol.

> On Jan. 29, 2016, 5:52 p.m., John Sirois wrote: > > jfarrell: I'm definitely going to wait on your opinion on this one. I'm > > viewing you as the expert in this bunch on what the [RFC > > 2119](https://www.ietf.org/rfc/rfc2119.txt) keywords are for these emails. I'll take silence as implic

Re: Review Request 43172: Add deprecated field storage backfill

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43172/#review117808 --- Fix it, then Ship it! src/main/java/org/apache/aurora/schedule

Re: Review Request 43172: Add deprecated field storage backfill

> On Feb. 4, 2016, 3:37 a.m., Stephan Erb wrote: > > During this whole odyssee, have we learned a thing which is not reflected > > here? > > https://github.com/apache/aurora/blob/master/docs/thrift-deprecation.md > > > > If yes, please write it down :-) I concur, but I'm fine with the doc upd

Re: Review Request 43172: Add deprecated field storage backfill

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43172/#review117791 --- During this whole odyssee, have we learned a thing which is not re