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

2014-02-25 Thread Suman Karumuri
On Feb. 25, 2014, 11:11 p.m., Kevin Sweeney wrote: src/main/java/org/apache/aurora/scheduler/base/Jobs.java, line 48 https://reviews.apache.org/r/17303/diff/1/?file=447775#file447775line48 This uses time so it should take a Clock instance for testing. Since we got rid of

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

2014-01-24 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17303/#review32720 --- src/main/thrift/org/apache/aurora/gen/api.thrift

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

2014-01-24 Thread Suman Karumuri
On Jan. 24, 2014, 4:44 p.m., Bill Farner wrote: src/main/thrift/org/apache/aurora/gen/api.thrift, line 187 https://reviews.apache.org/r/17303/diff/1/?file=447779#file447779line187 I think pre-computing these stats restricts the freedom of an API consumer from displaying them

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