Re: Review Request 52872: Used callback instead of `send()` for health status updates.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52872/ --- (Updated Nov. 25, 2016, 11:52 a.m.) Review request for mesos, Anand Mazumdar, Benjamin Mahler, Gastón Kleiman, and haosdent huang. Bugs: MESOS-6395 https://issues.apache.org/jira/browse/MESOS-6395 Repository: mesos Description --- Since HealthChecker is now used as a library only and does not live in a separate OS process, there is no need to use libprocess message sending for health status updates; a callback will do. Diffs (updated) - src/docker/executor.cpp eefbc0c2936eb93044c2d8a1b50155171f398562 src/health-check/health_checker.hpp 837d1358e418d21536da488e4a23cbfa41db6060 src/health-check/health_checker.cpp af5500be249c74a4a5e64bf38dea607173e2f998 src/launcher/default_executor.cpp f4e1ea4816fee9a2f6d59bb534933b66013f11cf src/launcher/executor.cpp ce0b199551447504bb95743df4ce9ec4a0443cd4 Diff: https://reviews.apache.org/r/52872/diff/ Testing --- See https://reviews.apache.org/r/52873/. Thanks, Alexander Rukletsov
Re: Review Request 52872: Used callback instead of `send()` for health status updates.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52872/#review156867 --- Ship it! Ship It! - Gastón Kleiman On Nov. 23, 2016, 12:52 a.m., Alexander Rukletsov wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52872/ > --- > > (Updated Nov. 23, 2016, 12:52 a.m.) > > > Review request for mesos, Anand Mazumdar, Benjamin Mahler, Gastón Kleiman, > and haosdent huang. > > > Bugs: MESOS-6395 > https://issues.apache.org/jira/browse/MESOS-6395 > > > Repository: mesos > > > Description > --- > > Since HealthChecker is now used as a library only and does not live > in a separate OS process, there is no need to use libprocess message > sending for health status updates; a callback will do. > > > Diffs > - > > src/docker/executor.cpp eefbc0c2936eb93044c2d8a1b50155171f398562 > src/health-check/health_checker.hpp > 837d1358e418d21536da488e4a23cbfa41db6060 > src/health-check/health_checker.cpp > af5500be249c74a4a5e64bf38dea607173e2f998 > src/launcher/default_executor.cpp f4e1ea4816fee9a2f6d59bb534933b66013f11cf > src/launcher/executor.cpp ce0b199551447504bb95743df4ce9ec4a0443cd4 > > Diff: https://reviews.apache.org/r/52872/diff/ > > > Testing > --- > > See https://reviews.apache.org/r/52873/. > > > Thanks, > > Alexander Rukletsov > >
Re: Review Request 52872: Used callback instead of `send()` for health status updates.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52872/ --- (Updated Nov. 23, 2016, 12:52 a.m.) Review request for mesos, Anand Mazumdar, Benjamin Mahler, Gastón Kleiman, and haosdent huang. Bugs: MESOS-6395 https://issues.apache.org/jira/browse/MESOS-6395 Repository: mesos Description --- Since HealthChecker is now used as a library only and does not live in a separate OS process, there is no need to use libprocess message sending for health status updates; a callback will do. Diffs (updated) - src/docker/executor.cpp eefbc0c2936eb93044c2d8a1b50155171f398562 src/health-check/health_checker.hpp 837d1358e418d21536da488e4a23cbfa41db6060 src/health-check/health_checker.cpp af5500be249c74a4a5e64bf38dea607173e2f998 src/launcher/default_executor.cpp f4e1ea4816fee9a2f6d59bb534933b66013f11cf src/launcher/executor.cpp ce0b199551447504bb95743df4ce9ec4a0443cd4 Diff: https://reviews.apache.org/r/52872/diff/ Testing --- See https://reviews.apache.org/r/52873/. Thanks, Alexander Rukletsov
Re: Review Request 52872: Used callback instead of `send()` for health status updates.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52872/#review156394 --- Ship it! Maybe add a TODO to replace the callback with a `process::Stream`? - Benjamin Mahler On Nov. 14, 2016, 10:30 a.m., Alexander Rukletsov wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52872/ > --- > > (Updated Nov. 14, 2016, 10:30 a.m.) > > > Review request for mesos, Anand Mazumdar, Benjamin Mahler, Gastón Kleiman, > and haosdent huang. > > > Bugs: MESOS-6395 > https://issues.apache.org/jira/browse/MESOS-6395 > > > Repository: mesos > > > Description > --- > > Since HealthChecker is now used as a library only and does not live > in a separate OS process, there is no need to use libprocess message > sending for health status updates; a callback will do. > > > Diffs > - > > src/docker/executor.cpp eefbc0c2936eb93044c2d8a1b50155171f398562 > src/health-check/health_checker.hpp > a1dc72493ff31b87068d5691f4d5b794392caf76 > src/health-check/health_checker.cpp > e2b32e2d57515202f547d12ba492ad8eb633641b > src/launcher/default_executor.cpp f4e1ea4816fee9a2f6d59bb534933b66013f11cf > src/launcher/executor.cpp ce0b199551447504bb95743df4ce9ec4a0443cd4 > > Diff: https://reviews.apache.org/r/52872/diff/ > > > Testing > --- > > See https://reviews.apache.org/r/52873/. > > > Thanks, > > Alexander Rukletsov > >
Re: Review Request 52872: Used callback instead of `send()` for health status updates.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52872/#review155942 --- Ship it! Ship It! - haosdent huang On Nov. 14, 2016, 10:30 a.m., Alexander Rukletsov wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52872/ > --- > > (Updated Nov. 14, 2016, 10:30 a.m.) > > > Review request for mesos, Anand Mazumdar, Benjamin Mahler, Gastón Kleiman, > and haosdent huang. > > > Bugs: MESOS-6395 > https://issues.apache.org/jira/browse/MESOS-6395 > > > Repository: mesos > > > Description > --- > > Since HealthChecker is now used as a library only and does not live > in a separate OS process, there is no need to use libprocess message > sending for health status updates; a callback will do. > > > Diffs > - > > src/docker/executor.cpp eefbc0c2936eb93044c2d8a1b50155171f398562 > src/health-check/health_checker.hpp > a1dc72493ff31b87068d5691f4d5b794392caf76 > src/health-check/health_checker.cpp > e2b32e2d57515202f547d12ba492ad8eb633641b > src/launcher/default_executor.cpp f4e1ea4816fee9a2f6d59bb534933b66013f11cf > src/launcher/executor.cpp ce0b199551447504bb95743df4ce9ec4a0443cd4 > > Diff: https://reviews.apache.org/r/52872/diff/ > > > Testing > --- > > See https://reviews.apache.org/r/52873/. > > > Thanks, > > Alexander Rukletsov > >
Re: Review Request 52872: Used callback instead of `send()` for health status updates.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52872/ --- (Updated Nov. 14, 2016, 10:30 a.m.) Review request for mesos, Anand Mazumdar, Benjamin Mahler, Gastón Kleiman, and haosdent huang. Bugs: MESOS-6395 https://issues.apache.org/jira/browse/MESOS-6395 Repository: mesos Description --- Since HealthChecker is now used as a library only and does not live in a separate OS process, there is no need to use libprocess message sending for health status updates; a callback will do. Diffs (updated) - src/docker/executor.cpp eefbc0c2936eb93044c2d8a1b50155171f398562 src/health-check/health_checker.hpp a1dc72493ff31b87068d5691f4d5b794392caf76 src/health-check/health_checker.cpp e2b32e2d57515202f547d12ba492ad8eb633641b src/launcher/default_executor.cpp f4e1ea4816fee9a2f6d59bb534933b66013f11cf src/launcher/executor.cpp ce0b199551447504bb95743df4ce9ec4a0443cd4 Diff: https://reviews.apache.org/r/52872/diff/ Testing --- See https://reviews.apache.org/r/52873/. Thanks, Alexander Rukletsov