Re: Review Request 64885: Made sure continuations using actor state are executed in actor context.

2018-01-02 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64885/#review194634 --- Ship it! Ship It! src/slave/containerizer/mesos/provisioner/p

Re: Review Request 64885: Made sure continuations using actor state are executed in actor context.

2018-01-02 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64885/#review194632 --- Ship it! Ship It! - Zhitao Li On Jan. 2, 2018, 10:38 a.m., B

Re: Review Request 64885: Made sure continuations using actor state are executed in actor context.

2018-01-02 Thread Benjamin Bannier
> On Jan. 2, 2018, 5:12 p.m., Zhitao Li wrote: > > src/slave/containerizer/mesos/provisioner/provisioner.cpp > > Line 568 (original), 568 (patched) > > > > > > I don't think the callback here uses any internal state

Re: Review Request 64885: Made sure continuations using actor state are executed in actor context.

2018-01-02 Thread Zhitao Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64885/#review194629 --- src/slave/containerizer/mesos/provisioner/provisioner.cpp Line 56

Review Request 64885: Made sure continuations using actor state are executed in actor context.

2018-01-02 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64885/ --- Review request for mesos, Gilbert Song and Zhitao Li. Repository: mesos Descr