-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66322/
-----------------------------------------------------------
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