---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32585/#review79904
---
src/master/master.hpp
On April 13, 2015, 1:47 p.m., Ben Mahler wrote:
src/master/master.hpp, lines 1100-1103
https://reviews.apache.org/r/32585/diff/4/?file=919351#file919351line1100
Did you look back at https://reviews.apache.org/r/19176/ when doing
this?
What was the motivation for
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32585/#review79799
---
Ship it!
Needs a quick rebase before committing.
- Adam B
On
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32585/#review79146
---
LGTM, assuming no significant rebases.
src/master/master.hpp
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32585/
---
(Updated April 7, 2015, 12:59 p.m.)
Review request for mesos, Adam B and
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32585/#review78496
---
src/slave/slave.cpp
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32585/
---
(Updated April 1, 2015, 3:34 p.m.)
Review request for mesos, Adam B and Niklas
On April 1, 2015, 5:28 a.m., Adam B wrote:
src/slave/slave.cpp, line 1043
https://reviews.apache.org/r/32585/diff/1-2/?file=908253#file908253line1043
Hmm... I was actually thinking that this line of code can be removed
now that you're always filling in the id in the FrameworkInfo
On April 1, 2015, 2:28 a.m., Adam B wrote:
src/slave/slave.cpp, line 1043
https://reviews.apache.org/r/32585/diff/1-2/?file=908253#file908253line1043
Hmm... I was actually thinking that this line of code can be removed
now that you're always filling in the id in the FrameworkInfo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32585/
---
(Updated March 31, 2015, 4:29 p.m.)
Review request for mesos, Adam B and
10 matches
Mail list logo