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

(Updated Oct. 12, 2017, 1:10 p.m.)


Review request for mesos and Benjamin Mahler.


Changes
-------

Updated benchmark results.


Bugs: MESOS-6971
    https://issues.apache.org/jira/browse/MESOS-6971


Repository: mesos


Description
-------

This changes `ProtobufProcess` to use arenas for creating messages.


Diffs (updated)
-----

  3rdparty/libprocess/include/process/protobuf.hpp 
e46a076b19de711900f9023ae5c53db45951b547 


Diff: https://reviews.apache.org/r/62901/diff/2/

Changes: https://reviews.apache.org/r/62901/diff/1-2/


Testing (updated)
-------

Ran benchmark
```
./benchmarks --gtest_filter=ProcessTest.Process_BENCHMARK_MessagePassing
```
before and after changes with the following results:
```
Without arenas
[ RUN      ] ProcessTest.Process_BENCHMARK_MessagePassing
Size: 4,        elapsed: 14.800662ms
Size: 10,       elapsed: 23.607056ms
Size: 34,       elapsed: 53.234174ms
Size: 64,       elapsed: 86.6773ms
Size: 154,      elapsed: 191.359858ms
Size: 304,      elapsed: 330.10146ms
Size: 605,      elapsed: 612.198306ms
Size: 1507,     elapsed: 1.515456025secs
Size: 3011,     elapsed: 3.006522946secs
Size: 4515,     elapsed: 4.532226307secs
Size: 6019,     elapsed: 6.045134458secs
Size: 9027,     elapsed: 9.077230263secs
Size: 12034,    elapsed: 12.156748523secs
Size: 15042,    elapsed: 15.199843166secs
Size: 30081,    elapsed: 30.394480817secs
[       OK ] ProcessTest.Process_BENCHMARK_MessagePassing (83252 ms)


With arenas
[ RUN      ] ProcessTest.Process_BENCHMARK_MessagePassing
Size: 4,        elapsed: 18.910807ms
Size: 10,       elapsed: 28.725021ms
Size: 34,       elapsed: 53.235732ms
Size: 64,       elapsed: 86.577933ms
Size: 154,      elapsed: 157.530026ms
Size: 304,      elapsed: 284.589934ms
Size: 605,      elapsed: 541.077226ms
Size: 1507,     elapsed: 1.311257359secs
Size: 3011,     elapsed: 2.591459371secs
Size: 4515,     elapsed: 3.879464983secs
Size: 6019,     elapsed: 5.168277919secs
Size: 9027,     elapsed: 8.265426958secs
Size: 12034,    elapsed: 11.087320011secs
Size: 15042,    elapsed: 13.854052262secs
Size: 30081,    elapsed: 28.361437857secs
[       OK ] ProcessTest.Process_BENCHMARK_MessagePassing (75702 ms)
```


Thanks,

Dmitry Zhuk

Reply via email to