mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Schwartzmeyer <and...@schwartzmeyer.com>
Subject Review Request 67900: Fixed compilation error with `DispatchProcess` code.
Date Thu, 12 Jul 2018 20:35:58 GMT

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

Review request for mesos, Benjamin Mahler, Joseph Wu, and Michael Park.


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


Repository: mesos


Description
-------

Per MESOS-8686 and MESOS-8915, the benchmark test
`ProcessTest.Process_BENCHMARK_DispatchDefer` had been disabled as the
`DispatchProcess` class no longer compiled with `/permissive-` using
the latest version of MSVC. The fix is thanks to PhoebeHui.


Diffs
-----

  3rdparty/libprocess/src/tests/benchmarks.cpp 604122a2f33d7abfab92631d9eac33b6753e846b 


Diff: https://reviews.apache.org/r/67900/diff/1/


Testing
-------

On Windows:

```
> .\3rdparty\libprocess\src\tests\benchmarks.exe
[==========] Running 5 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 5 tests from ProcessTest
[ RUN      ] ProcessTest.Process_BENCHMARK_ClientServer
Client 0: 102041 rpcs / sec
Client 1: 102041 rpcs / sec
Client 2: 100002 rpcs / sec
Client 3: 101012 rpcs / sec
Client 4: 100002 rpcs / sec
Client 5: 100002 rpcs / sec
Client 6: 101012 rpcs / sec
Client 7: 101012 rpcs / sec
Estimated Total: 761911 rpcs / sec
[       OK ] ProcessTest.Process_BENCHMARK_ClientServer (122 ms)
[ RUN      ] ProcessTest.Process_BENCHMARK_LargeNumberOfLinks
Elapsed: 88023us
[       OK ] ProcessTest.Process_BENCHMARK_LargeNumberOfLinks (291 ms)
[ RUN      ] ProcessTest.Process_BENCHMARK_ThroughputPerformance
Estimated Total: 6399307.850863
[       OK ] ProcessTest.Process_BENCHMARK_ThroughputPerformance (2347 ms)
[ RUN      ] ProcessTest.Process_BENCHMARK_DispatchDefer
Movable elapsed: 768.006300000000010ms
Copyable elapsed: 1.310028600000000secs
[       OK ] ProcessTest.Process_BENCHMARK_DispatchDefer (2082 ms)
[ RUN      ] ProcessTest.Process_BENCHMARK_ProtobufInstallHandler
Size:     4 bytes, throughput:   3445844 messages/s
Size:    10 bytes, throughput:   2384994 messages/s
Size:    34 bytes, throughput:   1347373 messages/s
Size:    64 bytes, throughput:    990755 messages/s
Size:   304 bytes, throughput:    328788 messages/s
Size:   605 bytes, throughput:    181178 messages/s
Size:  3011 bytes, throughput:     39931 messages/s
Size:  6019 bytes, throughput:     20242 messages/s
Size: 30081 bytes, throughput:      4039 messages/s
Size: 60160 bytes, throughput:      1952 messages/s
[       OK ] ProcessTest.Process_BENCHMARK_ProtobufInstallHandler (10028 ms)
[----------] 5 tests from ProcessTest (14873 ms total)

[----------] Global test environment tear-down
[==========] 5 tests from 1 test case ran. (14874 ms total)
[  PASSED  ] 5 tests.
```


Thanks,

Andrew Schwartzmeyer


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