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 Wed, 30 Dec 2015 20:47:21 GMT

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

(Updated Dec. 30, 2015, 8:47 p.m.)


Review request for mesos, Ben Mahler and Jie Yu.


Changes
-------

Jie correctly spotted that we can land into `escalationTimeout` via `killTask` and `shutdown`.
Not a good place to be. Removed the two states for now. Would put them back in a subsequent
review.


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


Repository: mesos


Description
-------

See summary.


Diffs (updated)
-----

  src/launcher/executor.cpp 09e7de6f2136ed9ca13e4822aeabcba2f7c5837d 

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