aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Zameer Manji <zma...@apache.org>
Subject Re: Review Request 59539: Add cluster state debug endpoint to Scheduler HTTP servlet
Date Wed, 24 May 2017 22:04:38 GMT

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


Ship it!




Ship It!

- Zameer Manji


On May 24, 2017, 1:51 p.m., David McLaughlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59539/
> -----------------------------------------------------------
> 
> (Updated May 24, 2017, 1:51 p.m.)
> 
> 
> Review request for Aurora, Santhosh Kumar Shanmugham, Stephan Erb, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> The struct exposed here is the one used to make preemption decisions - it has the entire
state of running tasks. It's a very handy summary structure to analyse the state of the current
scheduling results. Exposing it here to make it easier to do analysis on our test clusters
(currently have to process backups).
> 
> 
> Diffs
> -----
> 
>   src/main/java/org/apache/aurora/scheduler/http/JettyServerModule.java 53ebc0b60e60a8e1800f9e8b4ddc292bb44b4330

>   src/main/java/org/apache/aurora/scheduler/http/State.java PRE-CREATION 
>   src/main/resources/scheduler/assets/index.html 9bd22a8d5b1aebe7e19ebfd1077fb36f82ea6c36

>   src/test/java/org/apache/aurora/scheduler/http/StateTest.java PRE-CREATION 
> 
> 
> Diff: https://reviews.apache.org/r/59539/diff/2/
> 
> 
> Testing
> -------
> 
> Tests pass. Also ran in vagrant.
> 
> 
> Thanks,
> 
> David McLaughlin
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message