----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60008/ -----------------------------------------------------------
Review request for mesos, Alexander Rukletsov and Michael Park. Bugs: MESOS-5886 https://issues.apache.org/jira/browse/MESOS-5886 Repository: mesos Description ------- FUTURE_DISPATCH uses DispatchMatcher to figure out whether a processed DispatchEvent is the same the user is waiting for. Currently, we compare std::type_info of function pointers, which is not enough: different class methods with same signatures will be matched (see MESOS-5886 for an example). This patch adds value of pointer-to-member function in addition to std::type_info in DispatchEvent to uniquely identify class methods. Diffs ----- 3rdparty/libprocess/include/process/dispatch.hpp 3a0793888dc0df5e3ec31b06f47cd920c71e0db9 3rdparty/libprocess/include/process/event.hpp 8afe6266eb0dc5a17af35d79efb6bfdf9e6a0ee9 3rdparty/libprocess/include/process/gmock.hpp e9af943b39436f365fe687301febb5c7fbefffc4 3rdparty/libprocess/src/process.cpp 4ff7448d171f39dbb8cbb81dd9bed136ad43d62d 3rdparty/libprocess/src/tests/process_tests.cpp 38d787a083a5eb31e922d283f4b4bed2bd62eb0a Diff: https://reviews.apache.org/r/60008/diff/1/ Testing ------- 1. make check (mac os x 10.12, fedora 25) 2. internal CI NOTE: Test GroupTest.ConnectTimer is broken, bacause it uses FUTURE_DISPATCH on a wrong method (GroupProcess::expired), which has the same signature as a private method (GroupProcess::timedout). The later method is actually called, thus causing the error after applying this patch. Thanks, Andrei Budnik