> On May 25, 2018, 3:12 p.m., Benjamin Bannier wrote: > > 3rdparty/libprocess/src/grpc.cpp > > Lines 76 (patched) > > <https://reviews.apache.org/r/67157/diff/4/?file=2024986#file2024986line86> > > > > This seems almost like the libprocess equivalent of a throwing > > destructor. Could we trigger this e.g., if we fail a test `ASSERT`? > > > > Also see my comment regarding joining whenever `looper` goes out of > > scope.
Given that we never directly call `process::terminate` on a `RuntimeProcess`, but only indirectly terminate the process through `RuntimeProcess::terminate` which also sets `terminating`, this cannot fail at any case, including failed test assertions. I could remove this if this line looks weird to you. - Chun-Hung ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67157/#review203871 ----------------------------------------------------------- On May 17, 2018, 10:23 p.m., Chun-Hung Hsiao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67157/ > ----------------------------------------------------------- > > (Updated May 17, 2018, 10:23 p.m.) > > > Review request for mesos, Benjamin Bannier, Benjamin Mahler, Jie Yu, and > Zhitao Li. > > > Bugs: MESOS-8924 > https://issues.apache.org/jira/browse/MESOS-8924 > > > Repository: mesos > > > Description > ------- > > The refactoring does the following things: > 1. Manage the gRPC completion queue and the looper thread in the runtime > process to get rid of a lock in `Runtime::Data`. > 2. Move the computation of sending a request into the runtime process. > 3. Let libprocess manage the runtime process automatically instead of > managing its lifecycle in `Runtime::Data`. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/grpc.hpp > 321a46e19c69eafb24012bcef68bb8b0cc6aa436 > 3rdparty/libprocess/src/grpc.cpp a80bcb614ec96d92d21bc88a281d3208e86141a0 > > > Diff: https://reviews.apache.org/r/67157/diff/4/ > > > Testing > ------- > > sudo make check > > > Thanks, > > Chun-Hung Hsiao > >
