> On March 17, 2014, 10:26 p.m., Bill Farner wrote:
> > I started on this in parallel, and have a few additional changes i'd like 
> > you to include.  Please have a look at the tip commit on 
> > origin/wfarner/fix_jobs_page. (Convenience link: 
> > https://git-wip-us.apache.org/repos/asf?p=incubator-aurora.git;a=commitdiff;h=2ce304215b099687c3d2f89701a81707a11cca7f)
> > 
> > Also, please remove [BugFix] from the review summary and instead link the 
> > review against the appropriate bug.

Done. Thanks for taking a look.


- Suman


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


On March 17, 2014, 11:57 p.m., Suman Karumuri wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19323/
> -----------------------------------------------------------
> 
> (Updated March 17, 2014, 11:57 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Bill Farner.
> 
> 
> Bugs: AURORA-271
>     https://issues.apache.org/jira/browse/AURORA-271
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Added jobStats on role page. 
> 
> We have stopped counting recently failed task count which is used to display 
> the health badge. Since this badge is being removed from the new UI, removed 
> it from the old UI as well.
> 
> 
> Diffs
> -----
> 
>   build.gradle f38888beb00f69f40e81c1be80f5f6ade84720ae 
>   src/main/java/org/apache/aurora/scheduler/base/Jobs.java 
> 3f027032a785810bddfb3ff4dd4e472989c9493f 
>   src/main/java/org/apache/aurora/scheduler/http/SchedulerzRole.java 
> b3f3124a73b56e48339f89dbd3b1067527757d04 
>   
> src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java
>  ee81b494b9b4dff9839a9e32c9b81fa8ef898e1b 
>   src/main/python/apache/aurora/tools/java/thrift_wrapper_codegen.py 
> 3143c6afe8adeb18fac44e92f94b9860753c3eb0 
>   src/main/resources/org/apache/aurora/scheduler/http/schedulerzrole.st 
> 39c1d9527bff88ff191233629babfd6e6035d4e7 
>   src/test/java/org/apache/aurora/scheduler/base/JobsTest.java 
> 6524e2441d25de6573190866a8559be620961330 
> 
> Diff: https://reviews.apache.org/r/19323/diff/
> 
> 
> Testing
> -------
> 
> ./gradlew clean build
> gradlew run
> 
> 
> File Attachments
> ----------------
> 
> job stats
>   
> https://reviews.apache.org/media/uploaded/files/2014/03/17/65da4a77-4c82-46f8-93d1-34fd319ef60e__Screen_Shot_2014-03-17_at_3.13.36_PM.png
> 
> 
> Thanks,
> 
> Suman Karumuri
> 
>

Reply via email to