Re: Review Request 46759: Updated Master to deal with Accept Offer vs InverseOffer correctly.

2016-05-31 Thread Joris Van Remoortere
> On April 28, 2016, 7:24 p.m., Vinod Kone wrote: > > src/master/metrics.cpp, line 227 > > > > > > hmm...there is no "messages_accept_offers"? also, these are not really > > messages but calls. see discussion in

Re: Review Request 46759: Updated Master to deal with Accept Offer vs InverseOffer correctly.

2016-04-28 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46759/#review130976 --- src/master/master.cpp (line 3935)

Re: Review Request 46759: Updated Master to deal with Accept Offer vs InverseOffer correctly.

2016-04-27 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46759/#review130877 --- Patch looks great! Reviews applied: [46756, 46757, 46758, 46759]