You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@mesos.apache.org by Sergey Urbanovich <se...@gmail.com> on 2018/07/26 05:39:12 UTC

Review Request 68052: Discard the queue's promises in the destructor.

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

Review request for mesos and Jie Yu.


Repository: mesos


Description
-------

It fixes the case when we can get forever-pending futures after the
destruction of the queue.


Diffs
-----

  3rdparty/libprocess/include/process/queue.hpp c364b7dc831050bc1b23f669dc1ccc1f1255a9c8 
  3rdparty/libprocess/src/tests/queue_tests.cpp cc6b3d82ce5eb5ca16e2b9378dd5b1f28ac62fe8 


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


Testing
-------

3rdparty/libprocess/src/tests/libprocess-tests --verbose --gtest_filter="QueueTest*" --gtest_break_on_failure --gtest_repeat=1000


Thanks,

Sergey Urbanovich