You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@mesos.apache.org by Jiang Yan Xu <ya...@jxu.me> on 2013/08/06 20:05:58 UTC

Review Request 13307: Fixed a debug where we didn't catch all proc file read errors.

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

Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.


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


Repository: mesos-git


Description
-------

See summary. 

try/catch is used because we can get error message out of the stream than iostate checking.


Diffs
-----

  3rdparty/libprocess/3rdparty/stout/include/stout/proc.hpp 2fcc04b23cc4bb7ef6c7f02fdc7397f3bba84fc0 
  src/tests/reaper_tests.cpp d3e11072d5e8f5138210db566eec2d65942bcacd 

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


Testing
-------

make check

./bin/mesos-tests.sh --gtest_filter="ReaperTest.*" --gtest_repeat=10000 --gtest_break_on_failure --gtest_shuffle


Thanks,

Jiang Yan Xu