---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30774/
---
(Updated Feb. 18, 2015, 12:28 a.m.)
Review request for mesos, Adam B, Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31011/#review72924
---
Patch looks great!
Reviews applied: [31011]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30606/#review72923
---
Do we have test for this?
- Timothy Chen
On Feb. 18, 2015, 8:01
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30606/
---
(Updated Feb. 18, 2015, 12:01 a.m.)
Review request for mesos, Adam B, Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review72921
---
Patch looks great!
Reviews applied: [27760]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30774/
---
(Updated Feb. 18, 2015, 12:07 a.m.)
Review request for mesos, Adam B, Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30995/#review72927
---
First pass, found some minor typos/nits. Still considering whether
On Feb. 17, 2015, 6:39 p.m., Zameer Manji wrote:
Would it be possible to handle all default values in all protobufs in a
generic manner? For example the 'role' field of FrameworkInfo has the same
problem:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31114/
---
(Updated Feb. 18, 2015, 10:05 a.m.)
Review request for mesos, Ben Mahler, Till
On Feb. 17, 2015, 7:13 p.m., Vinod Kone wrote:
src/tests/master_tests.cpp, lines 1616-1624
https://reviews.apache.org/r/31114/diff/2/?file=866348#file866348line1616
Hmm. I don't understand the fix here.
First off, any expectations should be set *before* the
We generally don't have a lot of performance improvement work, but one
thing that came up recently for large clusters was:
https://issues.apache.org/jira/browse/MESOS-2353
On Tue, Feb 17, 2015 at 11:10 PM, Atri Sharma atri.j...@gmail.com wrote:
Hi all,
I am looking into starting hacking Mesos
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31114/#review72948
---
Patch looks great!
Reviews applied: [31114]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31011/
---
(Updated Feb. 18, 2015, 11:32 a.m.)
Review request for mesos and Till
On Feb. 16, 2015, 11:37 p.m., Adam B wrote:
src/authentication/cram_md5/authenticator.hpp, line 533
https://reviews.apache.org/r/27760/diff/14/?file=863170#file863170line533
Should this still `process::terminate(process, false)` if the short
term fix is now in
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review72932
---
Patch looks great!
Reviews applied: [27760]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31011/#review72943
---
Patch looks great!
Reviews applied: [31011]
All tests passed.
-
Hi Ben,
Thanks for your email.
If not performance, is there any other area where I could focus on?
On Wed, Feb 18, 2015 at 2:33 PM, Benjamin Mahler benjamin.mah...@gmail.com
wrote:
We generally don't have a lot of performance improvement work, but one
thing that came up recently for large
On Jan. 28, 2015, 12:33 p.m., Vinod Kone wrote:
src/master/master.cpp, lines 4193-4203
https://reviews.apache.org/r/27760/diff/12/?file=834181#file834181line4193
IIRC, sending an AuthenticationErrorMessage here causes the driver to
simply retry the authentication and thus fall
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30074/
---
(Updated Feb. 18, 2015, 11:36 a.m.)
Review request for mesos, Niklas Nielsen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review72942
---
src/master/master.cpp
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29497/#review72944
---
Ship it!
Ship It!
- Alexander Rukletsov
On Feb. 17, 2015, 10:55
I suggest you can also search for JIRA tickets on Mesos with the label
newbie to start out with.
https://issues.apache.org/jira/browse/MESOS-2314?jql=project%20%3D%20MESOS%20AND%20labels%20%3D%20newbie
Tim
On Wed, Feb 18, 2015 at 1:07 AM, Atri Sharma atri.j...@gmail.com wrote:
Hi Ben,
Thanks
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30074/#review72949
---
Ship it!
Ship It!
- Alexander Rukletsov
On Feb. 18, 2015, 10:36
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31011/#review72951
---
Ship it!
Ship It!
- Alexander Rukletsov
On Feb. 18, 2015, 11:32
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30074/#review72968
---
Patch looks great!
Reviews applied: [30074]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review72970
---
Patch looks great!
Reviews applied: [27760]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29507/#review72992
---
Let's get tests wired up before committing this :)
- Niklas
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29855/#review72996
---
Ship it!
Ship It!
- Niklas Nielsen
On Jan. 13, 2015, 10:55
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30545/#review72971
---
First pass, haven't looked at the tests yet. The main issue is that
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30736/#review72973
---
Sorry I'm just getting around to looking at this. Been busy with
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29406/
---
(Updated Feb. 18, 2015, 11:35 a.m.)
Review request for mesos, Benjamin Hindman
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31024/#review72975
---
Ship it!
nice tests.
src/slave/slave.hpp
On Feb. 18, 2015, 6:53 p.m., Vinod Kone wrote:
src/slave/slave.hpp, line 280
https://reviews.apache.org/r/31024/diff/1-3/?file=863795#file863795line280
s/flushTasks/runTasks/ so that we don't add a new verb flush?
Jie Yu wrote:
We already have 'runTask'. Sounds a little
On Jan. 28, 2015, 8:33 p.m., Vinod Kone wrote:
src/master/master.cpp, lines 4193-4203
https://reviews.apache.org/r/27760/diff/12/?file=834181#file834181line4193
IIRC, sending an AuthenticationErrorMessage here causes the driver to
simply retry the authentication and thus fall
On Feb. 18, 2015, 6:53 p.m., Vinod Kone wrote:
src/slave/slave.hpp, line 280
https://reviews.apache.org/r/31024/diff/1-3/?file=863795#file863795line280
s/flushTasks/runTasks/ so that we don't add a new verb flush?
We already have 'runTask'. Sounds a little confusing if we have both
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review72995
---
Patch looks great!
Reviews applied: [27760]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31114/#review72972
---
Ship it!
Ship It!
- Vinod Kone
On Feb. 18, 2015, 9:05 a.m.,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review72974
---
Patch looks great!
Reviews applied: [27760]
All tests passed.
-
On Feb. 18, 2015, 11:38 a.m., Niklas Nielsen wrote:
Let's get tests wired up before committing this :)
Sure thing. Adding tests in my subsequent patch where we will pass the master's
timeout values on to the slave. Will post that very soon.
- Adam
On Feb. 18, 2015, 8:16 p.m., Niklas Nielsen wrote:
src/slave/slave.cpp, line 3855
https://reviews.apache.org/r/30074/diff/8/?file=867823#file867823line3855
Max age of 0 seconds is hard to tell whether is a very unhealthy system
or no value present. Wonder if we should leave a
On Feb. 18, 2015, 9:16 p.m., Niklas Nielsen wrote:
src/slave/slave.cpp, line 3855
https://reviews.apache.org/r/30074/diff/8/?file=867823#file867823line3855
Max age of 0 seconds is hard to tell whether is a very unhealthy system
or no value present. Wonder if we should leave a
On Feb. 18, 2015, 8:16 p.m., Niklas Nielsen wrote:
src/slave/slave.cpp, line 3855
https://reviews.apache.org/r/30074/diff/8/?file=867823#file867823line3855
Max age of 0 seconds is hard to tell whether is a very unhealthy system
or no value present. Wonder if we should leave a
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30074/#review73003
---
The patch changed a bit since the first /state.json version.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review73011
---
Patch looks great!
Reviews applied: [27760]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review73021
---
Patch looks great!
Reviews applied: [27760]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31162/
---
Review request for mesos, Ben Mahler, Jie Yu, Niklas Nielsen, and Vinod Kone.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31162/#review73025
---
include/mesos/authentication/authentication.hpp
On Feb. 18, 2015, 5:42 p.m., Jie Yu wrote:
include/mesos/authentication/authentication.hpp, lines 25-27
https://reviews.apache.org/r/31162/diff/1/?file=868264#file868264line25
We don't do using namespace in headers.
Yes, but this was to avoid changing numerous cpp files to include
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review73089
---
Patch looks great!
Reviews applied: [27760]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30606/#review73096
---
3rdparty/libprocess/3rdparty/stout/include/stout/net.hpp
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31162/#review73087
---
Patch looks great!
Reviews applied: [31162]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review73093
---
Patch looks great!
Reviews applied: [27760]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29507/#review73085
---
Bad patch!
Reviews applied: [29507]
Failed command:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review73092
---
Patch looks great!
Reviews applied: [27760]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review73083
---
Patch looks great!
Reviews applied: [27760]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review72967
---
Patch looks great!
Reviews applied: [27760]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30774/
---
(Updated Feb. 18, 2015, 4:43 a.m.)
Review request for mesos, Adam B, Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31011/#review72959
---
Patch looks great!
Reviews applied: [31011]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30074/#review72957
---
Patch looks great!
Reviews applied: [30074]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review72963
---
Patch looks great!
Reviews applied: [27760]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30074/
---
(Updated Feb. 18, 2015, 4:01 p.m.)
Review request for mesos, Niklas Nielsen
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30074/
---
(Updated Feb. 18, 2015, 4:09 p.m.)
Review request for mesos, Niklas Nielsen
On Feb. 18, 2015, 10:42 p.m., Jie Yu wrote:
include/mesos/authentication/authentication.hpp, lines 25-27
https://reviews.apache.org/r/31162/diff/1/?file=868264#file868264line25
We don't do using namespace in headers.
Kapil Arya wrote:
Yes, but this was to avoid changing
A total of 63 proposals were submitted for #MesosCon[1], up
significantly from 24 submitted for last year’s conference. Similar to
last year, the MesosCon program committee is opening these proposals up
for community review/feedback to better-inform our decisions about what
should be included in
On Feb. 18, 2015, 5:42 p.m., Jie Yu wrote:
include/mesos/authentication/authentication.hpp, lines 25-27
https://reviews.apache.org/r/31162/diff/1/?file=868264#file868264line25
We don't do using namespace in headers.
Kapil Arya wrote:
Yes, but this was to avoid changing
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31162/
---
(Updated Feb. 18, 2015, 5:26 p.m.)
Review request for mesos, Ben Mahler, Jie
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29406/
---
(Updated Feb. 18, 2015, 11:30 p.m.)
Review request for mesos, Benjamin Hindman
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31162/#review73066
---
The high level comment is: you should only put the namespace hacks
On Feb. 18, 2015, 7:38 p.m., Niklas Nielsen wrote:
Let's get tests wired up before committing this :)
Adam B wrote:
Sure thing. Adding tests in my subsequent patch where we will pass the
master's timeout values on to the slave. Will post that very soon.
Can you do it in one patch?
On Feb. 19, 2015, 2:28 a.m., Jie Yu wrote:
src/tests/state_tests.cpp, lines 86-87
https://reviews.apache.org/r/31162/diff/1/?file=868284#file868284line86
typedef Registry::Slaves Slaves;
typedef Registry::Slave Slave;
This reverts the code to it's original state which is most
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28775/
---
(Updated Feb. 19, 2015, 12:05 a.m.)
Review request for mesos, Adam B, Benjamin
On Feb. 18, 2015, 2:42 p.m., Jie Yu wrote:
include/mesos/authentication/authentication.hpp, lines 25-27
https://reviews.apache.org/r/31162/diff/1/?file=868264#file868264line25
We don't do using namespace in headers.
Kapil Arya wrote:
Yes, but this was to avoid changing
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28775/#review73069
---
Patch looks great!
Reviews applied: [28775]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30972/
---
(Updated Feb. 18, 2015, 11:43 p.m.)
Review request for mesos, Benjamin
On Feb. 15, 2015, 4:55 p.m., Alexander Rukletsov wrote:
src/master/allocator/mesos/hierarchical.hpp, lines 436-437
https://reviews.apache.org/r/30911/diff/2/?file=863977#file863977line436
Is this still relevant?
I think so, did you find somewhere where we do this already?
On
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30973/
---
(Updated Feb. 18, 2015, 11:43 p.m.)
Review request for mesos, Benjamin Hindman
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30976/
---
(Updated Feb. 18, 2015, 11:44 p.m.)
Review request for mesos and Vinod Kone.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30971/
---
(Updated Feb. 18, 2015, 11:43 p.m.)
Review request for mesos, Benjamin Hindman
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30978/
---
(Updated Feb. 18, 2015, 11:44 p.m.)
Review request for mesos, Niklas Nielsen,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29406/#review73032
---
Patch looks great!
Reviews applied: [29526, 29527, 29528, 29529,
On Feb. 11, 2015, 12:08 a.m., Ben Mahler wrote:
It might be time to introduce os::stat and os::lstat wrappers to avoid
needing to introduce so many special case functions.
Bernd Mathiske wrote:
That would reduce portability.
Ben Mahler wrote:
Oh? Could you elaborate please?
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29748/
---
(Updated Feb. 19, 2015, 12:06 a.m.)
Review request for mesos, Adam B, Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31162/
---
(Updated Feb. 18, 2015, 7:59 p.m.)
Review request for mesos, Ben Mahler, Jie
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27760/#review73047
---
Patch looks great!
Reviews applied: [27760]
All tests passed.
-
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30978/#review73065
---
Patch looks great!
Reviews applied: [30970, 30971, 30972, 30973,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29507/
---
(Updated Feb. 18, 2015, 4:33 p.m.)
Review request for mesos and Niklas
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30015/#review73076
---
Nice, can you split the patches for `lstat` and `pathconf` please?
On Feb. 16, 2015, 11:43 p.m., Jie Yu wrote:
include/mesos/resources.hpp, lines 155-161
https://reviews.apache.org/r/30911/diff/2/?file=863975#file863975line155
I don't think we should remove these two helper functions as they are
quite convinent to use. (just like we still have
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30911/
---
(Updated Feb. 19, 2015, 2:54 a.m.)
Review request for mesos, Adam B, Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31038/
---
(Updated Feb. 19, 2015, 2:55 a.m.)
Review request for mesos, Adam B, Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30875/
---
(Updated Feb. 19, 2015, 2:55 a.m.)
Review request for mesos, Adam B, Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30361/
---
(Updated Feb. 19, 2015, 2:55 a.m.)
Review request for mesos, Adam B, Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30874/
---
(Updated Feb. 19, 2015, 2:55 a.m.)
Review request for mesos, Adam B, Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29856/
---
(Updated Feb. 19, 2015, 2:55 a.m.)
Review request for mesos, Adam B, Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31039/
---
(Updated Feb. 19, 2015, 2:55 a.m.)
Review request for mesos, Adam B, Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28698/
---
(Updated Feb. 19, 2015, 2:55 a.m.)
Review request for mesos, Adam B, Benjamin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29748/#review73079
---
Patch looks great!
Reviews applied: [29742, 30911, 30873, 28698,
97 matches
Mail list logo