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 https://reviews.apache.org/r/17205/diff/1/?file=435558#file435558line456 this one too Will do this in the next diff. Closing out this diff since I am blocked on these

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 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 https://reviews.apache.org/r/17088/diff/1/?file=430129#file430129line427 parens on lhs are unnecessary here. Fixed. On Jan. 23, 2014, 10:10 p.m., Kevin

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

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

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:

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

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