-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17303/
-----------------------------------------------------------
(Updated March 9, 2014, 10:17 a.m.)
Review request for Aurora, Kevin Sweeney and Bill Farner.
Changes
-------
Updated code per review comments.
Bugs: AURORA-64
https://issues.apache.org/jira/browse/AURORA-64
Repository: aurora
Description
-------
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
60b2259f21b598fa38bec5a590516cba2c07e1ac
src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java
4911c77a16c486fabfead7ad2f84ee95423ecd93
src/main/thrift/org/apache/aurora/gen/api.thrift
d72b28c3378a651a8cff49216c1435ce7aee5977
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
9712f30a71be206fbf417198d0af673b45e0281e
src/test/java/org/apache/aurora/scheduler/thrift/aop/ForwardingThrift.java
a5fcbd465b5e07e23b24524e060cea304f102492
src/test/resources/org/apache/aurora/gen/api.thrift.md5
2308ba8da96197d41040ba772ea871003615698a
Diff: https://reviews.apache.org/r/17303/diff/
Testing
-------
gradle clean build
gradle run to test local UI.
Thanks,
Suman Karumuri
