incubator-mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jiang Yan Xu" <>
Subject Re: Review Request: Changed reaper to use Future as the notification mechanism. The notification is now sent only if the pid is explicitly registered via the monitor() call.
Date Fri, 31 May 2013 18:43:29 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated May 31, 2013, 6:43 p.m.)

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


Addressed Vinod's comments.


Rebased against master. Previous review received "shipit"s but got lost.

Diffs (updated)

  src/slave/cgroups_isolator.hpp 7b8270d0b0f94a71da12bc123d39b44c40c3f7ed 
  src/slave/cgroups_isolator.cpp 9b3a3a5dfec27a119fdd47a88f016e21470eb88d 
  src/slave/process_isolator.hpp 9875f4a6e8e109e31ad390fbd7a84d03ad747190 
  src/slave/process_isolator.cpp d4f7b7603452e7226189899aca7e3c94e537829f 
  src/slave/reaper.hpp 09844d8d47b143ee369e0c82b19d65a774df4a90 
  src/slave/reaper.cpp 1d826d1b5fe1048b03662a1e9d12b18371e5c88f 
  src/tests/reaper_tests.cpp fbb6066652c0f36c0a8fd9b0c4dac7e0240a5210 



make check.


Jiang Yan Xu

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