mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <>
Subject Review Request 64885: Made sure continuations using actor state are executed in actor context.
Date Tue, 02 Jan 2018 10:38:16 GMT

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

Review request for mesos, Gilbert Song and Zhitao Li.

Repository: mesos


In `e273efe6976434858edb85bbcf367a02e963a467` we introduced layer
checkpointing to the provisioner, but did not make sure that
continutations making use of internal state of the provisioner actor
installed on futures where always executed in the actor's context.
This was problematic as continuations could be executed while actor
state was changing (data races), or after the the actor had terminated
(use after free).

In this patch we instead defer execution of continutions to the actor
owning the data.

This issue was identified with the clang-tidy `mesos-this-capture`


  src/slave/containerizer/mesos/provisioner/provisioner.cpp 61e771872ba2c3f5ecabbe085db102433c7bac3f



`make check`


Benjamin Bannier

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