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 Mon, 16 Mar 2015 23:06:27 GMT

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

(Updated March 16, 2015, 11:06 p.m.)

Review request for mesos, Niklas Nielsen and Vinod Kone.


Addressed comments, updated description.

Bugs: MESOS-2499

Repository: mesos

Description (updated)

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

Diffs (updated)

  src/slave/slave.cpp 0f99e4efb8fa2b96f120a3e49191158ca0364c06 
  src/tests/slave_tests.cpp a975305430097a8295b4b155e8448572c12bde22 



make check (Mac OS 10.9.5, CentOS 7.0.1406)


Alexander Rukletsov

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