mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Maged Michael <maged.mich...@gmail.com>
Subject Review Request 43144: Added env var to set upper bound on number of worker threads.
Date Wed, 03 Feb 2016 20:57:06 GMT

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

Review request for mesos.


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

Added env var to set upper bound on number of worker threads.


Repository: mesos


Description (updated)
-------

Added env var to set upper bound on number of worker threads.


Diffs (updated)
-----

  3rdparty/libprocess/src/process.cpp d8a74d7637d20c81f384e974e4fdeba22effb437 

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


Testing (updated)
-------

Invocations of mesos-master, mesos-slave, and mesos-tests with the LIBPROCESS_MAX_WORKER_THREADS
set to the values "1", "7", "9", "1000", "0", "-1", and "abc" on a 4-core x86_64 Ubuntu 14.04
system.

The results were as expected. 1, 7, 8, 8, 8, 8, and 8 worker threads were created, respectively.
The last 3 tests generated warnings, e.g., "LIBPROCESS_MAX_WORKER_THREADS=abc is invalid".


Thanks,

Maged Michael


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