Re: Review Request 59680: Added the 'LocalResourceProvider' structure.

2017-06-01 Thread Jie Yu
> On June 1, 2017, 12:02 a.m., Jie Yu wrote: > > src/master/master.hpp > > Lines 291 (patched) > > > > > > Why both `LocalResourceProvider` and `ResourceProvider`? I'd prefer we > > don't have

Re: Review Request 59680: Added the 'LocalResourceProvider' structure.

2017-06-01 Thread Jan Schlicht
> On June 1, 2017, 2:02 a.m., Jie Yu wrote: > > src/master/master.hpp > > Lines 291 (patched) > > > > > > Why both `LocalResourceProvider` and `ResourceProvider`? I'd prefer we > > don't have

Re: Review Request 59680: Added the 'LocalResourceProvider' structure.

2017-05-31 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59680/#review176557 --- src/master/master.hpp Lines 291 (patched)

Re: Review Request 59680: Added the 'LocalResourceProvider' structure.

2017-05-31 Thread Jan Schlicht
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59680/ --- (Updated May 31, 2017, 2:43 p.m.) Review request for mesos, Benjamin Bannier

Review Request 59680: Added the 'LocalResourceProvider' structure.

2017-05-31 Thread Jan Schlicht
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59680/ --- Review request for mesos, Benjamin Bannier and Jie Yu. Repository: mesos