----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68132/#review206768 -----------------------------------------------------------
Bad patch! Reviews applied: [68132, 68131] Failed command: python support/apply-reviews.py -n -r 68131 Error: The support scripts will be upgraded to Python 3 by July 1st. Make sure to install Python 3.6 on your machine before. 2018-08-01 23:05:18 URL:https://reviews.apache.org/r/68131/diff/raw/ [6814/6814] -> "68131.patch" [1] error: patch failed: src/tests/master_benchmarks.cpp:482 error: src/tests/master_benchmarks.cpp: patch does not apply Full log: https://builds.apache.org/job/Mesos-Reviewbot/22988/console - Mesos Reviewbot On July 31, 2018, 5:24 p.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68132/ > ----------------------------------------------------------- > > (Updated July 31, 2018, 5:24 p.m.) > > > Review request for mesos, Benno Evers and Benjamin Mahler. > > > Bugs: MESOS-9122 > https://issues.apache.org/jira/browse/MESOS-9122 > > > Repository: mesos > > > Description > ------- > > With this patch handlers for '/state' requests are not scheduled > directly after authorization, but are accumulated and then scheduled > for later parallel processing. > > This approach allows, if there are N '/state' requests in the Master's > mailbox and T is the request response time, to block the Master actor > only once for time O(T) instead of blocking it for time N*T prior to > this patch. > > This batching technique reduces both the time Master is spending > answering '/state' requests and the average request response time > in presence of multiple requests in the Master's mailbox. However, > for seldom '/state' requests that don't accumulate in the Master's > mailbox, the response time might increase due to an added trip > through the mailbox. > > > Diffs > ----- > > src/master/http.cpp 6947031da3ce3523408d69d6dac60551a39a4601 > src/master/master.hpp 0353d550308816f219aedb6afe15c643fc8bb340 > src/master/master.cpp 2af976f7ea7f81d4b06a45ce13286dbd61b9b144 > > > Diff: https://reviews.apache.org/r/68132/diff/1/ > > > Testing > ------- > > `make check` on Mac OS 10.13.5 and various Linux distros. > > Run `MasterStateQueryLoad_BENCHMARK_Test.v0State` benchmark. > > Average improvement without optimization: 62%–70%. > Average improvement with optimization: 17%–62%. > > **[No batching, no > optimization](https://dobianchi.files.wordpress.com/2011/11/no-barrique-no-berlusconi.jpg?w=638)** > ``` > Test setup: 100 agents with a total of 10000 running tasks and 10000 > completed tasks; 10 '/state' and '/flags' requests will be sent with 200ms > interval > Launching 10 '/state' requests in background > Launching 10 '/flags' requests > '/flags' response on average took 1.102349605secs, 10 responses are in > [2.662342ms, 2.143755433secs] > '/state' response on average took 1.549122019secs, 10 responses are in > [494.278454ms, 2.633971927secs] > > Test setup: 1000 agents with a total of 100000 running tasks and 100000 > completed tasks; 10 '/state' and '/flags' requests will be sent with 200ms > interval > Launching 10 '/state' requests in background > Launching 10 '/flags' requests > '/flags' response on average took 18.436968137secs, 10 responses are in > [2.578238ms, 33.210561732secs] > '/state' response on average took 23.916379537secs, 10 responses are in > [5.170660597secs, 43.008091744secs] > ``` > > **With batching but no optimization** > ``` > Test setup: 100 agents with a total of 10000 running tasks and 10000 > completed tasks; 10 '/state' and '/flags' requests will be sent with 200ms > interval > Launching 10 '/state' requests in background > Launching 10 '/flags' requests > '/flags' response on average took 417.211022ms, 10 responses are in > [4.066901ms, 728.045442ms] > '/state' response on average took 830.351291ms, 10 responses are in > [459.033455ms, 1.208880892secs] > > Test setup: 1000 agents with a total of 100000 running tasks and 100000 > completed tasks; 10 '/state' and '/flags' requests will be sent with 200ms > interval > Launching 10 '/state' requests in background > Launching 10 '/flags' requests > '/flags' response on average took 5.439950928secs, 10 responses are in > [3.246906ms, 9.343994388secs] > '/state' response on average took 16.764607823secs, 10 responses are in > [4.980333091secs, 18.461983916secs] > ``` > > **No batching but `-O3` optimization** > ``` > Test setup: 100 agents with a total of 10000 running tasks and 10000 > completed tasks; 10 '/state' and '/flags' requests will be sent with 200ms > interval > Launching 10 '/state' requests in background > Launching 10 '/flags' requests > '/flags' response on average took 2.396221ms, 10 responses are in > [1.628583ms, 2.816639ms] > '/state' response on average took 113.469574ms, 10 responses are in > [104.218099ms, 134.477062ms] > > Test setup: 1000 agents with a total of 100000 running tasks and 100000 > completed tasks; 10 '/state' and '/flags' requests will be sent with 200ms > interval > Launching 10 '/state' requests in background > Launching 10 '/flags' requests > '/flags' response on average took 3.892615876secs, 10 responses are in > [2.480517ms, 7.630934838secs] > '/state' response on average took 5.205245306secs, 10 responses are in > [1.578161651secs, 8.789315237secs] > ``` > > **Batching and `-O3` optimization** > ``` > Test setup: 100 agents with a total of 10000 running tasks and 10000 > completed tasks; 10 '/state' and '/flags' requests will be sent with 200ms > interval > Launching 10 '/state' requests in background > Launching 10 '/flags' requests > '/flags' response on average took 1.973573ms, 10 responses are in > [1.221193ms, 2.694713ms] > '/state' response on average took 113.331551ms, 10 responses are in > [102.593397ms, 142.028555ms] > > Test setup: 1000 agents with a total of 100000 running tasks and 100000 > completed tasks; 10 '/state' and '/flags' requests will be sent with 200ms > interval > Launching 10 '/state' requests in background > Launching 10 '/flags' requests > '/flags' response on average took 1.475842691secs, 10 responses are in > [2.437217ms, 3.815589561secs] > '/state' response on average took 4.742303751secs, 10 responses are in > [4.047655443secs, 6.00752698secs] > ``` > > > Thanks, > > Alexander Rukletsov > >