mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Steve Niemitz <sniem...@twitter.com>
Subject Re: Review Request 41049: New python lib with only the executor driver.
Date Tue, 08 Mar 2016 18:15:24 GMT

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

(Updated March 8, 2016, 6:15 p.m.)


Review request for mesos, Till Toenshoff and Vinod Kone.


Changes
-------

oops, some extra flags I had been using to debug with snuck into that last diff, reverted.


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


Repository: mesos


Description
-------

New python lib with only the executor driver.

This patch produces a new python egg, mesos.executor, which contains only the code needed
to create a MesosExecutorDriver.  By doing so, the linker can remove unused code in libmesos_no_3rdparty.a,
and therefor not include any external dependencies in the resulting _mesos.so.


Diffs (updated)
-----

  configure.ac a20382e8d425eb297492a6e6c2c75ea59be097c2 
  src/Makefile.am a41e95ddeb838fdebf4ced953c4a29181916e261 
  src/python/executor/setup.py.in PRE-CREATION 
  src/python/executor/src/mesos/__init__.py PRE-CREATION 
  src/python/executor/src/mesos/executor/__init__.py PRE-CREATION 
  src/python/executor/src/mesos/executor/module.cpp PRE-CREATION 
  src/python/native/ext_modules.py.in eb93864733713dddad66141c6b8b6cd895f41484 
  src/python/native/setup.py.in 49ed61293281f65d6597470ce3697326ac769032 
  src/python/native/src/mesos/native/__init__.py 226f94357cd81b700706edd68c94de461647fa1b

  src/python/native/src/mesos/native/mesos_executor_driver_impl.hpp  
  src/python/native/src/mesos/native/mesos_executor_driver_impl.cpp 7838a07ac12034a5eed2f459bd11e5e0a07e94de

  src/python/native/src/mesos/native/mesos_scheduler_driver_impl.hpp  
  src/python/native/src/mesos/native/mesos_scheduler_driver_impl.cpp f8be49bf5feb6675c3c8e6a1df75a5e079d09fcc

  src/python/native/src/mesos/native/module.hpp 2cf7b57077d10c071bd4b8f36e2cb43041fc168d 
  src/python/native/src/mesos/native/module.cpp 92373611b8d3c54d8af97b55064074bd423726ee 
  src/python/native/src/mesos/native/proxy_executor.hpp  
  src/python/native/src/mesos/native/proxy_executor.cpp 706f417b547bb9878346902b2a2f98b1afade5ad

  src/python/native/src/mesos/native/proxy_scheduler.hpp  
  src/python/native/src/mesos/native/proxy_scheduler.cpp 8afb3380d99b4dc9c4f7b3d926f4a5b1d6e94c20

  src/python/native_common/ext_modules.py.in PRE-CREATION 
  src/python/scheduler/setup.py.in PRE-CREATION 
  src/python/scheduler/src/mesos/__init__.py PRE-CREATION 
  src/python/scheduler/src/mesos/scheduler/__init__.py PRE-CREATION 

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


Testing
-------

On CentOS 7, GCC 4.8.5:
- make distcheck
- Tested with aurora executor
- Made sure mesos.scheduler still worked
- Made sure mesos.native still worked


Thanks,

Steve Niemitz


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