mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mesos Reviewbot Windows <revi...@mesos.apache.org>
Subject Re: Review Request 62901: Used protobuf arenas for creating messages in ProtobufProcess.
Date Thu, 12 Oct 2017 13:41:11 GMT

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



FAIL: Failed to apply the dependent review: 62898.

Failed command: `python.exe .\support\apply-reviews.py -n -r 62898`

All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/62901

Relevant logs:

- [apply-review-62898-stdout.log](http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/62901/logs/apply-review-62898-stdout.log):

```
error: patch failed: 3rdparty/libprocess/include/process/protobuf.hpp:128
error: 3rdparty/libprocess/include/process/protobuf.hpp: patch does not apply
```

- Mesos Reviewbot Windows


On Oct. 12, 2017, 1:10 p.m., Dmitry Zhuk wrote:
> 
> -----------------------------------------------------------
> 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.
> 
> 
> Bugs: MESOS-6971
>     https://issues.apache.org/jira/browse/MESOS-6971
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> This changes `ProtobufProcess` to use arenas for creating messages.
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/include/process/protobuf.hpp e46a076b19de711900f9023ae5c53db45951b547

> 
> 
> Diff: https://reviews.apache.org/r/62901/diff/2/
> 
> 
> Testing
> -------
> 
> 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
> 
>


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