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 60437: Add timing metrics in MesosCallbackHandler for backward compatibility.
Date Mon, 26 Jun 2017 17:54:33 GMT

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


Ship it!




Ship It!

- Zameer Manji


On June 26, 2017, 10:37 a.m., Kai Huang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/60437/
> -----------------------------------------------------------
> 
> (Updated June 26, 2017, 10:37 a.m.)
> 
> 
> Review request for Aurora, David McLaughlin, Santhosh Kumar, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Add timing metrics in MesosCallbackHandler for backward compatibility.
> 
> We have an internal performance dashboard that reads the following timing metrics:
> ```
> scheduler_resource_offers,
> scheduler_framework_message,
> scheduler_status_update
> ```
> Currently, these timing metrics merely reside in MesosSchedulerImpl class. If we were
to enable the Mesos HTTP API in aurora(switch from __MesosSchedulerImpl__ to __VersionedMesosSchedulerImpl__),
we will lose the timing metrics in MesosSchedulerImpl. This CR migrates the missing timing
metrics from MesosSchedulerImpl to MesosCallbackHandler, so that the above timing metrics
will be visible after enabling Mesos HTTP API.
> 
> 
> Diffs
> -----
> 
>   src/main/java/org/apache/aurora/scheduler/mesos/MesosCallbackHandler.java 772a04cbad99eb088105822f41b0758f47f7915a

>   src/main/java/org/apache/aurora/scheduler/mesos/MesosSchedulerImpl.java c3a34d2ae51a02247a7e6176dec0f04bddc0270d

>   src/test/java/org/apache/aurora/scheduler/mesos/MesosCallbackHandlerTest.java b955d61bbed2a6f4cb7f2581ebec398d637f0895

> 
> 
> Diff: https://reviews.apache.org/r/60437/diff/1/
> 
> 
> Testing
> -------
> 
> ./build-support/jenkins/build.sh
> 
> 
> Thanks,
> 
> Kai Huang
> 
>


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