> On Dec. 11, 2017, 4:38 p.m., Greg Mann wrote: > > src/status_update_manager/status_update_manager_process.hpp > > Line 112 (original), 112 (patched) > > <https://reviews.apache.org/r/64472/diff/1/?file=1911816#file1911816line112> > > > > Let's make the process ID "offer-operation-status-update-manager". > > Gaston Kleiman wrote: > I made the process ID configurable, see: > https://reviews.apache.org/r/64509/
Thanks!!! :) - Greg ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64472/#review193297 ----------------------------------------------------------- On Dec. 11, 2017, 9:36 p.m., Gaston Kleiman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64472/ > ----------------------------------------------------------- > > (Updated Dec. 11, 2017, 9:36 p.m.) > > > Review request for mesos and Greg Mann. > > > Repository: mesos > > > Description > ------- > > This patch adds to the log and error messages the type of status update > handled by the instance of `StatusUpdateManagerProcess`. > > > Diffs > ----- > > src/status_update_manager/offer_operation.cpp > f66690eadc0d7b16cc7de1b518904db2cab7ef82 > src/status_update_manager/status_update_manager_process.hpp > 1ac64410594134ce3e709d396d0a74a3d0aa2007 > > > Diff: https://reviews.apache.org/r/64472/diff/2/ > > > Testing > ------- > > `GLOG_v=1 bin/mesos-tests.sh > --gtest_filter="*OfferOperationStatusUpdateManagerTest*" --verbose` on > GNU/Linux > > > Thanks, > > Gaston Kleiman > >