Re: Review Request 17205: Moved readonly thrift API calls into ReadOnlyScheduler.

2014-01-23 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17205/#review32638 --- Ship it! src/main/thrift/org/apache/aurora/gen/api.thrift

Re: Review Request 17205: Moved readonly thrift API calls into ReadOnlyScheduler.

2014-01-23 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17205/#review32654 --- Ship it! Ship It! - Kevin Sweeney On Jan. 22, 2014, 1:02 p.m., S

Re: Review Request 17205: Moved readonly thrift API calls into ReadOnlyScheduler.

2014-01-23 Thread Suman Karumuri
> On Jan. 23, 2014, 6:17 p.m., Bill Farner wrote: > > src/main/thrift/org/apache/aurora/gen/api.thrift, line 456 > > > > > > this one too Will do this in the next diff. Closing out this diff since I am blocked on the

Re: Review Request 17088: Cache hashCode in generated immutable classes.

2014-01-23 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17088/#review32660 --- src/main/python/apache/aurora/tools/java/thrift_wrapper_codegen.py

Re: Review Request 17088: Cache hashCode in generated immutable classes.

2014-01-23 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17088/#review32672 --- Ship it! Ship It! - Kevin Sweeney On Jan. 18, 2014, 11:19 a.m.,

Re: Review Request 17088: Cache hashCode in generated immutable classes.

2014-01-23 Thread Bill Farner
> On Jan. 23, 2014, 10:10 p.m., Kevin Sweeney wrote: > > src/main/python/apache/aurora/tools/java/thrift_wrapper_codegen.py, line 427 > > > > > > parens on lhs are unnecessary here. Fixed. > On Jan. 23, 2014, 10:10

Re: Review Request 17088: Cache hashCode in generated immutable classes.

2014-01-23 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17088/ --- (Updated Jan. 23, 2014, 10:29 p.m.) Review request for Aurora, Kevin Sweeney an

Review Request 17270: Bridge framework for client v2.

2014-01-23 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17270/ --- Review request for Aurora, Bill Farner and Brian Wickman. Bugs: aurora-76 h

Review Request 17293: s/thermos_observe/thermos_observer/ in README.md

2014-01-23 Thread David Robinson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17293/ --- Review request for Aurora, Jonathan Boulle and Brian Wickman. Repository: auror

Re: Review Request 17293: s/thermos_observe/thermos_observer/ in README.md

2014-01-23 Thread David Robinson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17293/ --- (Updated Jan. 24, 2014, 1:28 a.m.) Review request for Aurora, Jonathan Boulle a

Review Request 17303: Updated getJobs API to return task stats and latest task config

2014-01-23 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17303/ --- Review request for Aurora, Kevin Sweeney and Bill Farner. Bugs: AURORA-64 h