----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66322/#review200092 -----------------------------------------------------------
Patch looks great! Reviews applied: [66118, 66119, 66120, 65679, 66126, 66143, 66322] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose --disable-libtool-wrappers' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot On March 28, 2018, 1:15 a.m., Meng Zhu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66322/ > ----------------------------------------------------------- > > (Updated March 28, 2018, 1:15 a.m.) > > > Review request for mesos, Chun-Hung Hsiao, Greg Mann, and Jie Yu. > > > Bugs: MESOS-8741 > https://issues.apache.org/jira/browse/MESOS-8741 > > > Repository: mesos > > > Description > ------- > > Adding item to sequence is realized by dispatching > `add()` to the sequence actor. However, this could > race with the sequence actor termination. > > This patch fixes this by enqueueing the terminate > message at the end of the message queue. > > Also removed the clock settle in the test `DiscardAll`. > As the processing of the messages are now guaranteed > to happen before the actor termination. > > Also added comments to clarify the onDiscard propagation. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/sequence.hpp > b4d7593221bf225a9e53e3b7f94e45624400078a > 3rdparty/libprocess/src/tests/sequence_tests.cpp > 43911b6c7a4220a4b8ea1baca191035355817e7b > > > Diff: https://reviews.apache.org/r/66322/diff/1/ > > > Testing > ------- > > make check > > After modifying test `DiscardAll`, it fails with the old implementation. > After modifying the `inject` flag, it passes. > > > Thanks, > > Meng Zhu > >