Re: Review Request 29015: Suppressing duplicate error messages.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29015/#review65117 --- The changes look good to me. Can you please update the testing porti

Re: Review Request 29005: Changing log level for the RequestLogger.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29005/ --- (Updated Dec. 15, 2014, 8:50 p.m.) Review request for Aurora, Joshua Cohen and

Review Request 29058: Extract mechanism for determining pex version so it can be used for cases other than client version.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29058/ --- Review request for Aurora, David McLaughlin, Kevin Sweeney, and Zameer Manji. R

Re: Review Request 29058: Extract mechanism for determining pex version so it can be used for cases other than client version.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29058/#review65119 --- Ship it! Ship It! - Zameer Manji On Dec. 15, 2014, 12:54 p.m., J

Re: Review Request 29058: Extract mechanism for determining pex version so it can be used for cases other than client version.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29058/#review65120 --- src/main/python/apache/aurora/client/cli/__init__.py

Re: Review Request 29058: Extract mechanism for determining pex version so it can be used for cases other than client version.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29058/#review65122 --- src/main/python/apache/aurora/client/cli/__init__.py

Re: Review Request 29058: Extract mechanism for determining pex version so it can be used for cases other than client version.

> On Dec. 15, 2014, 9:18 p.m., Kevin Sweeney wrote: > > src/main/python/apache/aurora/client/cli/__init__.py, line 73 > > > > > > How about constraining the exceptions thrown here in the library? > > Better, why not ju

Re: Review Request 29058: Extract mechanism for determining pex version so it can be used for cases other than client version.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29058/ --- (Updated Dec. 15, 2014, 9:52 p.m.) Review request for Aurora, David McLaughlin,

Re: Review Request 29058: Extract mechanism for determining pex version so it can be used for cases other than client version.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29058/#review65126 --- Ship it! +1, please file a followup ticket to include the actual ve

Re: Review Request 29058: Extract mechanism for determining pex version so it can be used for cases other than client version.

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

Re: Review Request 29058: Extract mechanism for determining pex version so it can be used for cases other than client version.

> On Dec. 15, 2014, 9:54 p.m., Kevin Sweeney wrote: > > +1, please file a followup ticket to include the actual version in the > > "version" output. https://issues.apache.org/jira/browse/AURORA-970 - Joshua --- This is an automatically

Re: Review Request 29058: Extract mechanism for determining pex version so it can be used for cases other than client version.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29058/#review65131 --- Ship it! Ship It! - David McLaughlin On Dec. 15, 2014, 9:52 p.m.

Re: Review Request 28731: Implemented TaskScheduler benchmarks.

> On Dec. 11, 2014, 10:28 p.m., Bill Farner wrote: > > Please document new classes (excluding fakes, since they're obvious). Done. > On Dec. 11, 2014, 10:28 p.m., Bill Farner wrote: > > src/jmh/java/org/apache/aurora/benchmark/SchedulingBenchmarks.java, line 96 > >

Re: Review Request 28731: Implemented TaskScheduler benchmarks.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28731/ --- (Updated Dec. 15, 2014, 10:15 p.m.) Review request for Aurora, Kevin Sweeney an

Re: Review Request 28617: Implemented offer filtering for tasks with static vetoes.

> On Dec. 12, 2014, 10:28 a.m., Stephan Erb wrote: > > src/main/java/org/apache/aurora/scheduler/async/OfferQueue.java, line 295 > > > > > > staticallyBannedOffers is only filled but never cleared. Looks like we > > w

Re: Review Request 28731: Implemented TaskScheduler benchmarks.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28731/#review65140 --- Master (52efc93) is green with this patch. ./build-support/jenkins

Re: Review Request 28617: Implemented offer filtering for tasks with static vetoes.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28617/ --- (Updated Dec. 15, 2014, 11:12 p.m.) Review request for Aurora, Kevin Sweeney an

Re: Review Request 29058: Extract mechanism for determining pex version so it can be used for cases other than client version.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29058/#review65151 --- This is on apache master. Feel free to close this review. - David M

Re: Review Request 28971: Begin dismantling client v1.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28971/#review65152 --- docs/client-commands.md

Re: Review Request 28913: Remove SANDBOX_DELETED state.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28913/#review65159 --- Ship it! src/test/java/org/apache/aurora/scheduler/base/JobsTest.j