Re: Review Request 60867: Added a method in the agent exposing an executor's allocated resources.

2017-07-17 Thread Benjamin Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60867/#review180776 --- Ship it! Ship It! - Benjamin Mahler On July 17, 2017, 12:09

Re: Review Request 60867: Added a method in the agent exposing an executor's allocated resources.

2017-07-17 Thread Andrei Budnik
> On July 14, 2017, 10:16 p.m., Benjamin Mahler wrote: > > src/slave/slave.hpp > > Lines 796-797 (original), 798-799 (patched) > > > > > > IIUC, this was supposed to represent the resources allocated the > >

Re: Review Request 60867: Added a method in the agent exposing an executor's allocated resources.

2017-07-17 Thread Andrei Budnik
> On July 14, 2017, 10:35 p.m., Benjamin Mahler wrote: > > src/slave/slave.cpp > > Lines 7816-7818 (patched) > > > > > > Also, you don't want to include terminated tasks :) Fixed. - Andrei

Re: Review Request 60867: Added a method in the agent exposing an executor's allocated resources.

2017-07-17 Thread Andrei Budnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60867/ --- (Updated July 17, 2017, 12:09 p.m.) Review request for mesos, Benjamin Mahler