mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jie Yu" <>
Subject Re: Review Request 17476: Added a Sequence abstraction.
Date Fri, 28 Feb 2014 22:47:23 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Feb. 28, 2014, 10:47 p.m.)

Review request for mesos and Benjamin Hindman.


BenM's comments.

Repository: mesos-git


See summary.

One think I haven't done yet is to make it accepts '_Defer' types. (Just curious why Future.onAny
can accept defer without doing something special).

Also, we can get rid of the SequencerProcess if we use a mutex to protect 'last'. Not sure
which one is better.

The discard semantics here is a bit tricky. Basically, I wanna support discarding a single
callback without affecting other callbacks. Also, when the Sequencer object is deleted, I
wanna discard all the pending callbacks.

Diffs (updated)

  3rdparty/libprocess/ a7d199f 
  3rdparty/libprocess/include/process/sequence.hpp PRE-CREATION 
  3rdparty/libprocess/src/tests/sequence_tests.cpp PRE-CREATION 



make check

repeated 1000 times


Jie Yu

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