mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gilbert Song <songzihao1...@gmail.com>
Subject Re: Review Request 52714: Added optional 'Type' to createExecutorInfo().
Date Fri, 04 Nov 2016 00:33:52 GMT

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

(Updated Nov. 3, 2016, 5:33 p.m.)


Review request for mesos, Anand Mazumdar, Jie Yu, Joris Van Remoortere, and Vinod Kone.


Repository: mesos


Description
-------

This patch made 2 changes to the helper because of the changes from
supporting nested containers:
1. Added an optional 'Type' in ExecutorInfo.
2. Make the parameter 'command' as optional since the CommandInfo
   became optional in 'ExecutorInfo' protobuf.


Diffs (updated)
-----

  src/tests/mesos.hpp 45dba233d9db39ead5f562e8090d07b5e0c6d42d 

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


Testing
-------

make check


Thanks,

Gilbert Song


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