Re: Review Request 23330: replace 143 kB favicon with 318 byte version

2014-07-07 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23330/#review47430 --- Ship it! Ship It! - Kevin Sweeney On July 7, 2014, 6:18 p.m., Da

Re: Review Request 23329: remove embedded jquery

2014-07-07 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/#review47428 --- src/main/python/apache/thermos/observer/http/templates/filebrowse.t

Review Request 23330: replace 143 kB favicon with 318 byte version

2014-07-07 Thread David Robinson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23330/ --- Review request for Aurora, David McLaughlin and Brian Wickman. Bugs: AURORA-577

Review Request 23329: remove embedded jquery

2014-07-07 Thread David Robinson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23329/ --- Review request for Aurora, David McLaughlin and Brian Wickman. Bugs: AURORA-578

Re: Review Request 23317: AURORA-565:update top level README with instructions on bootstrapping and running initial test

2014-07-07 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23317/#review47426 --- Ship it! Ship It! - Kevin Sweeney On July 7, 2014, 12:40 p.m., J

Re: Review Request 22457: Improve aurora "job diff" command.

2014-07-07 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22457/#review47423 --- It's been a week since the last time I pinged this review, and still

Review Request 23317: AURORA-565:update top level README with instructions on bootstrapping and running initial test

2014-07-07 Thread Jake Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23317/ --- Review request for Aurora, Kevin Sweeney and Bill Farner. Bugs: AURORA-565

Review Request 23316: Documentation for task health checking and graceful shutdown.

2014-07-07 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23316/ --- Review request for Aurora and Kevin Sweeney. Bugs: AURORA-574 https://issue

Re: Review Request 23315: s/Unsecure/Insecure/

2014-07-07 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23315/#review47406 --- Ship it! Ship It! - Kevin Sweeney On July 7, 2014, 11:28 a.m., B

Review Request 23315: s/Unsecure/Insecure/

2014-07-07 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23315/ --- Review request for Aurora and Kevin Sweeney. Repository: aurora Description -

Re: Review Request 22983: AURORA-278 - Remove binaries from source RC

2014-07-07 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22983/#review47403 --- Ship it! Ship It! - Kevin Sweeney On June 25, 2014, 1:18 p.m., J

Re: Review Request 23188: Adding getPendingReason RPC to expose scheduling vetos in the UI/client.

2014-07-07 Thread Bill Farner
> On July 2, 2014, 11:11 p.m., Bill Farner wrote: > > src/main/thrift/org/apache/aurora/gen/api.thrift, line 414 > > > > > > What's the motivation for including the job key? I would assume that > > the caller either