-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17303/
-----------------------------------------------------------

(Updated Feb. 26, 2014, 4:18 a.m.)


Review request for Aurora, Kevin Sweeney and Bill Farner.


Changes
-------

Added getJobSummary API per code review.
Addressed review feedback.


Summary (updated)
-----------------

Added getJobSummary API


Bugs: AURORA-64
    https://issues.apache.org/jira/browse/AURORA-64


Repository: aurora


Description (updated)
-------

Added getJobSummary API so it can be used by the role and role/environment page 
in the UI.
Refactored code from SchedulerzRole and SchedulerzRoleTest into relevant 
classes so it can be used by the UI and the thrift API.
Added tests for new code.
Moved populateJobConfig call into ReadOnlyScheduler.


Diffs (updated)
-----

  src/main/java/org/apache/aurora/scheduler/base/Jobs.java PRE-CREATION 
  src/main/java/org/apache/aurora/scheduler/base/Tasks.java 
d9cb886ef333e108d5d5f86043ac80e450689894 
  src/main/java/org/apache/aurora/scheduler/http/SchedulerzRole.java 
25ba7da5f8bbe5416f41bb0b14850beb84392cc7 
  
src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java 
dccae15a2c529025c9bb6a6f7a0220779ca4f9a1 
  src/main/thrift/org/apache/aurora/gen/api.thrift 
cd60f47bf34b4a634004e2ad9eadad37aa1556bb 
  src/test/java/org/apache/aurora/scheduler/base/JobsTest.java PRE-CREATION 
  src/test/java/org/apache/aurora/scheduler/base/TaskTestUtil.java PRE-CREATION 
  src/test/java/org/apache/aurora/scheduler/base/TasksTest.java PRE-CREATION 
  src/test/java/org/apache/aurora/scheduler/http/SchedulerzRoleTest.java 
912be189583419e7201e45650d18cd24a6a5a35b 
  
src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java
 4a2d39d8b25c4a6b161c47d6ba7068d74f8a60e0 
  src/test/java/org/apache/aurora/scheduler/thrift/aop/ForwardingThrift.java 
1edc0d7b224cc477ea6e8873e76ee8c70c6b4d50 
  src/test/resources/org/apache/aurora/gen/api.thrift.md5 
fafb5100443482e662db453429c5259f2ab80ae5 

Diff: https://reviews.apache.org/r/17303/diff/


Testing
-------

gradle clean build
gradle run to test local UI.


Thanks,

Suman Karumuri

Reply via email to