Re: Review Request 33154: Added reason metrics for slave removals.

2015-04-17 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33154/#review80444 --- src/master/master.hpp

Re: Review Request 33154: Added reason metrics for slave removals.

2015-04-16 Thread Ben Mahler
> On April 16, 2015, 4 p.m., Vinod Kone wrote: > > src/master/master.cpp, line 4465 > > > > > > Is this optional because there are other removals that we don't have > > metrics for? the ones in exited() and reregiste

Re: Review Request 33154: Added reason metrics for slave removals.

2015-04-16 Thread Ben Mahler
> On April 16, 2015, 4 p.m., Vinod Kone wrote: > > src/master/master.cpp, line 4465 > > > > > > Is this optional because there are other removals that we don't have > > metrics for? the ones in exited() and reregiste

Re: Review Request 33154: Added reason metrics for slave removals.

2015-04-16 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33154/#review80339 --- src/master/master.cpp

Review Request 33154: Added reason metrics for slave removals.

2015-04-13 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33154/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-2485 https://issues.ap