mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gastón Kleiman <gas...@mesosphere.com>
Subject Review Request 52879: Updated the way in which the default and command executors load options.
Date Fri, 14 Oct 2016 13:38:14 GMT

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

Review request for mesos, Alexander Rukletsov, Anand Mazumdar, Jie Yu, and Jiang Yan Xu.


Repository: mesos


Description
-------

Updated the command and the default executor so that they use only
`stout::flags` to load config options.

They used to use a mix of `stout::flags` and `os::getenv`.


Diffs
-----

  src/launcher/default_executor.cpp af4a97f7de5f2157aa65fdab742455b0683c40a4 
  src/launcher/executor.cpp 3e95d6029bea0ce6e0dfb39c24b795fe98d90d13 

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


Testing
-------

`make check`


Thanks,

Gastón Kleiman


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