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

(Updated Jan. 28, 2017, 1:11 p.m.)


Review request for Aurora, David McLaughlin and Stephan Erb.


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


Repository: aurora


Description (updated)
-------

This patch introduces a number of stats that measure the workload generated by 
Thrift server requests.

Current Thrift server stats expose the number and timing of requests received 
by the server. However, they fail to reflect the size of the requests. This is 
limiting us in having an accurate view of the workload handled by the 
scheduler. For example, every call to `restartShards()` is recorded as one 
event despite the fact that a request might only restart one shard while 
another request might seek to restart 1K shards.


Diffs
-----

  
src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java 
16b1b52f8691d978a9ec1bf7aa0c9716b3484cf0 
  
src/main/java/org/apache/aurora/scheduler/thrift/aop/ThriftStatsExporterInterceptor.java
 d57f910d8f9bbe5c24aec960e88d03702bc353da 
  src/main/java/org/apache/aurora/scheduler/thrift/aop/ThriftWorkload.java 
PRE-CREATION 
  
src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java
 b28cd2489a52041a8e7e53f298fad8d8cd29406f 
  
src/test/java/org/apache/aurora/scheduler/thrift/aop/ThriftStatsExporterInterceptorTest.java
 9c40ec51c28c8c57365dc21c3cd7391a3894784c 

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


Testing
-------

```
curl 192.168.33.7:8081/vars | grep thrift_workload
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 41334    0 41334    0     0  3695k      0 --:--:-- --:--:-- --:--:-- 4036k
thrift_workload_addInstances 0
thrift_workload_createJob 0
thrift_workload_createOrUpdateCronTemplate 0
thrift_workload_drainHosts 0
thrift_workload_endMaintenance 0
thrift_workload_getConfigSummary 0
thrift_workload_getJobSummary 0
thrift_workload_getJobUpdateDetails 0
thrift_workload_getJobUpdateSummaries 0
thrift_workload_getJobs 0
thrift_workload_getPendingReason 0
thrift_workload_getRoleSummary 0
thrift_workload_getTaskStatus 0
thrift_workload_getTasksWithoutConfigs 0
thrift_workload_killTasks 0
thrift_workload_maintenanceStatus 0
thrift_workload_restartShards 0
thrift_workload_rewriteConfigs 0
thrift_workload_startJobUpdate 0
thrift_workload_startMaintenance 0
```

```
./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh

...

*** OK (All tests passed) ***

mesos-master start/running, process 2359
+ RETCODE=0
+ restore_netrc
+ mv /home/vagrant/.netrc.bak /home/vagrant/.netrc
+ true
Connection to 127.0.0.1 closed.

real    28m58.389s
user    0m1.508s
sys     0m0.820s
```


Thanks,

Mehrdad Nurolahzade

Reply via email to