mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexander Rukletsov" <>
Subject Re: Review Request 32130: Ensured TaskStatus::source field is set for executor status updates.
Date Tue, 17 Mar 2015 21:54:13 GMT

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

(Updated March 17, 2015, 9:54 p.m.)

Review request for mesos, Niklas Nielsen and Vinod Kone.


Tested on gcc4.4

Bugs: MESOS-2499

Repository: mesos


A status update originating from executor should have the TaskStatus::source field set to
TaskStatus::SOURCE_EXECUTOR. Set this field in the slave to be future proof (a future where
there will be no executor driver). Previous code has a bug and updated a copy of the update
that was not forwarded. Add some checks for source and reason for status updates in existing

NOTE: Passing argument by value instead of `const&` is not fully consistent with our codebase,
but not entirely alien; it also documents our intention. This approach should be fine, since
`install()` deduces argument types from the return values of getter functions of protobuf


  src/slave/slave.hpp 989832f8783d07d0702b30f0a68b8c383b57c621 
  src/slave/slave.cpp 0f99e4efb8fa2b96f120a3e49191158ca0364c06 
  src/tests/slave_tests.cpp a975305430097a8295b4b155e8448572c12bde22 


Testing (updated)

make check (Mac OS 10.9.5, CentOS 7.0.1406, g++-4.4 @ Ubuntu 14.04)


Alexander Rukletsov

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