Re: Review Request 33317: Upgrade to pystachio 0.8.0

2015-04-17 Thread Bill Farner
> On April 17, 2015, 6:22 p.m., Bill Farner wrote: > > Can you include a test case proving that AURORA-1268 is fixed? > > Brian Wickman wrote: > Shouldn't that be in the reapplication of 27a602d2? It would be nice to tease apart the bug fix and the new feature. I'm fine if the bug fix com

Re: Review Request 33317: Upgrade to pystachio 0.8.0

2015-04-17 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33317/#review80500 --- Ship it! Master (b18dc44) is green with this patch. ./build-suppo

Re: Review Request 33317: Upgrade to pystachio 0.8.0

2015-04-17 Thread Brian Wickman
> On April 17, 2015, 6:22 p.m., Bill Farner wrote: > > Can you include a test case proving that AURORA-1268 is fixed? Shouldn't that be in the reapplication of 27a602d2? - Brian --- This is an automatically generated e-mail. To reply, v

Re: Review Request 33317: Upgrade to pystachio 0.8.0

2015-04-17 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33317/#review80496 --- Can you include a test case proving that AURORA-1268 is fixed? - Bi

Review Request 33317: Upgrade to pystachio 0.8.0

2015-04-17 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33317/ --- Review request for Aurora, Brian Brazil and Joshua Cohen. Bugs: AURORA-1268

Re: Review Request 32597: Improving async preemptor efficiency.

2015-04-17 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32597/#review80480 --- src/main/java/org/apache/aurora/scheduler/async/preemptor/PendingTa