----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59680/#review176557 -----------------------------------------------------------
src/master/master.hpp Lines 291 (patched) <https://reviews.apache.org/r/59680/#comment249943> Why both `LocalResourceProvider` and `ResourceProvider`? I'd prefer we don't have `LocalResourceProvider` top level, and rely on some field in `ResourceProvider` to tell - Jie Yu On May 31, 2017, 12:43 p.m., Jan Schlicht wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59680/ > ----------------------------------------------------------- > > (Updated May 31, 2017, 12:43 p.m.) > > > Review request for mesos, Benjamin Bannier and Jie Yu. > > > Bugs: MESOS-7592 > https://issues.apache.org/jira/browse/MESOS-7592 > > > Repository: mesos > > > Description > ------- > > A local resource provider is a resource provider that is associated > with an agent. Hence its lifetime is also tied to the lifetime of an > agent. The 'LocalResourceProvider' structure stores the ID of the > associated agent as well as the UUID that is used for registration. > > > Diffs > ----- > > src/master/master.hpp 89d0790fd5fea59e74276f462581fe0073594732 > src/master/master.cpp 14007e08f509446005423e223d5dd76a70744e27 > > > Diff: https://reviews.apache.org/r/59680/diff/1/ > > > Testing > ------- > > make check > > > Thanks, > > Jan Schlicht > >
