mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Anand Mazumdar" <mazumdar.an...@gmail.com>
Subject Re: Review Request 40998: Added CHECK's for various executor states in Command Executor to verify if messages are delivered in order.
Date Mon, 07 Dec 2015 23:19:14 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40998/
-----------------------------------------------------------

(Updated Dec. 7, 2015, 11:19 p.m.)


Review request for mesos and Ben Mahler.


Changes
-------

Changes based on feedback from BenM around not using an unused variable `executorInfo` but
introduing an `State` enum.


Summary (updated)
-----------------

Added CHECK's for various executor states in Command Executor to verify if messages are delivered
in order.


Bugs: MESOS-3851
    https://issues.apache.org/jira/browse/MESOS-3851


Repository: mesos


Description (updated)
-------

See summary.


Diffs (updated)
-----

  src/launcher/executor.cpp f90ea01131e6fa28e42f0c00a317d66a49f81ffa 

Diff: https://reviews.apache.org/r/40998/diff/


Testing
-------

make check


Thanks,

Anand Mazumdar


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