Re: Review Request 66309: Externalize creation of resource provider manager backing storage.

2018-04-10 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66309/ --- (Updated April 10, 2018, 2:07 p.m.) Review request for mesos, Jie Yu and Jan

Re: Review Request 66309: Externalize creation of resource provider manager backing storage.

2018-04-10 Thread Benjamin Bannier
> On March 28, 2018, 3:30 p.m., Jan Schlicht wrote: > > src/resource_provider/registrar.hpp > > Lines 68-69 (original), 70-71 (patched) > > > > > > I don't like the semantics of this: > > A `state::Storage`

Re: Review Request 66309: Externalize creation of resource provider manager backing storage.

2018-03-28 Thread Jan Schlicht
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66309/#review200105 --- src/resource_provider/registrar.hpp Lines 28 (patched)

Review Request 66309: Externalize creation of resource provider manager backing storage.

2018-03-27 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66309/ --- Review request for mesos, Jie Yu and Jan Schlicht. Bugs: MESOS-8735