mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alexander Rukletsov <>
Subject Re: Review Request 64070: Ensured executor adapter propagates error and shutdown messages.
Date Wed, 20 Dec 2017 20:16:42 GMT

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

(Updated Dec. 20, 2017, 8:16 p.m.)

Review request for mesos, Andrei Budnik, Anand Mazumdar, Armand Grillet, and Vinod Kone.

Summary (updated)

Ensured executor adapter propagates error and shutdown messages.

Repository: mesos

Description (updated)

Prior to this patch, if an error, kill, or shutdown occurred during
subscription / registration with the agent, it was not propagated back
to the executor if the v0_v1 executor adapter was used. This happened
because the adapter did not call the `connected` callback until after
successful registration and hence the executor did not even try to
send the `SUBSCRIBE` call, without which the adapter did not send any
events to the executor.

A fix is to call the `connected` callback if an error occurred or
shutdown / kill event arrived before the executor had subscribed.

Diffs (updated)

  src/executor/v0_v1executor.cpp 61d591993e6388ba3b4d64a3bdb63c3a3513fbeb 





Alexander Rukletsov

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