Review Request 18966: change the datatype of cpushare from size_t to uint64_t because The actual type of size_t is platform-dependent .

2014-03-11 Thread Archana kumari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18966/ --- Review request for mesos. Repository: mesos-git Description --- I

[jira] [Commented] (MESOS-1076) Add namespacing to cgroups to enforce the expected structure in slave/containerizer/isolators/cgroups/cpushare.cpp

2014-03-11 Thread Archana kumari (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1076?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930007#comment-13930007 ] Archana kumari commented on MESOS-1076: --- I am trying to upload a patch for this

Re: Review Request 18966: change the datatype of cpushare from size_t to uint64_t because The actual type of size_t is platform-dependent .

2014-03-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18966/#review36762 --- Patch looks great! Reviews applied: [18966] All tests passed. -

Re: Review Request 18966: change the datatype of cpushare from size_t to uint64_t because The actual type of size_t is platform-dependent .

2014-03-11 Thread Archana kumari
On March 11, 2014, 7:14 a.m., Mesos ReviewBot wrote: Patch looks great! Reviews applied: [18966] All tests passed. So should I ship it now? - Archana --- This is an automatically generated e-mail. To reply, visit:

Review Request 19024: Fixed a TODO in linux/cgroups.hpp

2014-03-11 Thread Archana kumari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19024/ --- Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.

Re: Review Request 19024: Fixed a TODO in linux/cgroups.hpp

2014-03-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19024/#review36766 --- Bad patch! Reviews applied: [19024] Failed command: make -j3

Re: Review Request 19024: Fixed a TODO in linux/cgroups.hpp

2014-03-11 Thread Archana kumari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19024/ --- (Updated March 11, 2014, 9:37 a.m.) Review request for mesos, Benjamin

Re: Review Request 19024: Fixed a TODO in linux/cgroups.hpp

2014-03-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19024/#review36768 --- Bad patch! Reviews applied: [19024] Failed command: make -j3

Re: Review Request 18946: Moved JNI code to separate library

2014-03-11 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18946/ --- (Updated March 11, 2014, 12:32 p.m.) Review request for mesos, Adam B, Ben

Re: Review Request 18946: Moved JNI code to separate library

2014-03-11 Thread Till Toenshoff
On March 10, 2014, 9:06 p.m., Timothy St. Clair wrote: So I looked over the patch, and it appears there will be a new install target but it doesn't appear to be shared or versioned? e.g. (no) -version-info 0:0:0 -release $(PACKAGE_VERSION) -shared Is there a reason? Clear

Re: Review Request 18946: Moved JNI code to separate library

2014-03-11 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18946/ --- (Updated March 11, 2014, 12:42 p.m.) Review request for mesos, Adam B, Ben

Re: Review Request 18946: Moved JNI code to separate library

2014-03-11 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18946/ --- (Updated March 11, 2014, 12:48 p.m.) Review request for mesos, Adam B, Ben

Re: Review Request 18946: Moved JNI code to separate library

2014-03-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18946/#review36779 --- Patch looks great! Reviews applied: [18946] All tests passed. -

Error occurs while installing mesos

2014-03-11 Thread HUO Jing
Hi, I met an error while installing mesos. When I run 'make' command, it shows: make[2]: ../3rdparty/libprocess/3rdparty/protobuf-2.4.1/src/protoc: Command not found make[2]: *** [../3rdparty/libprocess/3rdparty/protobuf-2.4.1/src/google/protobuf/descriptor.pb.h] Error 127 make[2]: Leaving

[jira] [Commented] (MESOS-1031) Command executor should support signal escalation.

2014-03-11 Thread Timothy St. Clair (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930445#comment-13930445 ] Timothy St. Clair commented on MESOS-1031: -- On [~bmahler]'s suggestion I looked

[jira] [Comment Edited] (MESOS-1031) Command executor should support signal escalation.

2014-03-11 Thread Timothy St. Clair (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930445#comment-13930445 ] Timothy St. Clair edited comment on MESOS-1031 at 3/11/14 3:18 PM:

[jira] [Commented] (MESOS-1031) Command executor should support signal escalation.

2014-03-11 Thread Niklas Quarfot Nielsen (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930473#comment-13930473 ] Niklas Quarfot Nielsen commented on MESOS-1031: --- Only using the process

Re: Review Request 18946: Moved JNI code to separate library

2014-03-11 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18946/#review36782 --- Ship it! Ship It! - Niklas Nielsen On March 11, 2014, 5:48

Re: Review Request 19024: Fixed a TODO in linux/cgroups.hpp

2014-03-11 Thread Archana kumari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19024/ --- (Updated March 11, 2014, 3:45 p.m.) Review request for mesos, Benjamin

[jira] [Commented] (MESOS-1031) Command executor should support signal escalation.

2014-03-11 Thread Timothy St. Clair (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930475#comment-13930475 ] Timothy St. Clair commented on MESOS-1031: -- That's why I mentioned PID

Re: Review Request 19024: Fixed a TODO in linux/cgroups.hpp

2014-03-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19024/#review36783 --- Bad patch! Reviews applied: [19024] Failed command: make -j3

Re: Review Request 18946: Moved JNI code to separate library

2014-03-11 Thread Timothy St. Clair
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18946/#review36785 --- Ship it! Ship It! - Timothy St. Clair On March 11, 2014, 12:48

[jira] [Commented] (MESOS-1031) Command executor should support signal escalation.

2014-03-11 Thread Niklas Quarfot Nielsen (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930495#comment-13930495 ] Niklas Quarfot Nielsen commented on MESOS-1031: --- Really like namespaces. We

Re: Review Request 18966: change the datatype of cpushare from size_t to uint64_t because The actual type of size_t is platform-dependent .

2014-03-11 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18966/#review36793 --- src/slave/containerizer/isolators/cgroups/cpushare.cpp

[jira] [Commented] (MESOS-1076) Add namespacing to cgroups to enforce the expected structure in slave/containerizer/isolators/cgroups/cpushare.cpp

2014-03-11 Thread Timothy St. Clair (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1076?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930551#comment-13930551 ] Timothy St. Clair commented on MESOS-1076: -- Did you run:

Re: Review Request 18966: change the datatype of cpushare from size_t to uint64_t because The actual type of size_t is platform-dependent .

2014-03-11 Thread Niklas Nielsen
On March 11, 2014, 12:14 a.m., Mesos ReviewBot wrote: Patch looks great! Reviews applied: [18966] All tests passed. Archana kumari wrote: So should I ship it now? Ship its are given by your reviewers :) A project member will commit the patch when it is ready:

Re: Review Request 19009: Added an in-memory Storage implementation.

2014-03-11 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19009/#review36794 --- src/state/in_memory.hpp

Re: Review Request 19008: Moved LevelDBStorageProcess and ZooKeeperStorageProcess out of the header files.

2014-03-11 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19008/#review36796 --- Ship it! Ship It! - Dominic Hamon On March 10, 2014, 8:44 p.m.,

Re: Review Request 18966: change the datatype of cpushare from size_t to uint64_t because The actual type of size_t is platform-dependent .

2014-03-11 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18966/#review36797 --- Ship it! Modulo Dominic's comment. - Niklas Nielsen On March

[jira] [Commented] (MESOS-1031) Command executor should support signal escalation.

2014-03-11 Thread Timothy St. Clair (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930565#comment-13930565 ] Timothy St. Clair commented on MESOS-1031: -- What *deployment platforms* are you

Re: Review Request 18946: Moved JNI code to separate library

2014-03-11 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18946/#review36800 --- Ship it!

Re: Review Request 19009: Added an in-memory Storage implementation.

2014-03-11 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19009/#review36801 --- src/state/in_memory.cpp

Re: Review Request 19007: Added log implementation for state storage.

2014-03-11 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19007/#review36798 --- src/state/log.cpp https://reviews.apache.org/r/19007/#comment67989

Re: Review Request 19003: Fixed JSON to protobuf parsing for enums.

2014-03-11 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19003/#review36802 --- 3rdparty/libprocess/3rdparty/stout/include/stout/protobuf.hpp

Re: Error occurs while installing mesos

2014-03-11 Thread Vinod Kone
Can you tell us your setup (OS, gcc version) and all the commands you used to run the build? On Tue, Mar 11, 2014 at 7:45 AM, HUO Jing huoj...@ihep.ac.cn wrote: Hi, I met an error while installing mesos. When I run 'make' command, it shows: make[2]:

[jira] [Commented] (MESOS-1055) Add explicit to single argument constructors

2014-03-11 Thread Bernd Mathiske (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1055?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930578#comment-13930578 ] Bernd Mathiske commented on MESOS-1055: --- https://reviews.apache.org/r/18823/ Add

Re: Review Request 18946: Moved JNI code to separate library

2014-03-11 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18946/ --- (Updated March 11, 2014, 4:59 p.m.) Review request for mesos and Vinod Kone.

[jira] [Commented] (MESOS-1031) Command executor should support signal escalation.

2014-03-11 Thread Niklas Quarfot Nielsen (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930586#comment-13930586 ] Niklas Quarfot Nielsen commented on MESOS-1031: --- I was referring to Mac OS X

[jira] [Created] (MESOS-1079) Mesos python egg build failure on OS X Mavericks (Xcode 5.1)

2014-03-11 Thread Till Toenshoff (JIRA)
Till Toenshoff created MESOS-1079: - Summary: Mesos python egg build failure on OS X Mavericks (Xcode 5.1) Key: MESOS-1079 URL: https://issues.apache.org/jira/browse/MESOS-1079 Project: Mesos

[jira] [Commented] (MESOS-1031) Command executor should support signal escalation.

2014-03-11 Thread Timothy St. Clair (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930678#comment-13930678 ] Timothy St. Clair commented on MESOS-1031: -- +1 I think as long as there is a

Re: Review Request 18966: change the datatype of cpushare from size_t to uint64_t because The actual type of size_t is platform-dependent .

2014-03-11 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18966/#review36820 --- src/slave/containerizer/isolators/cgroups/cpushare.cpp

[jira] [Comment Edited] (MESOS-1031) Command executor should support signal escalation.

2014-03-11 Thread Timothy St. Clair (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930678#comment-13930678 ] Timothy St. Clair edited comment on MESOS-1031 at 3/11/14 6:06 PM:

[jira] [Comment Edited] (MESOS-1031) Command executor should support signal escalation.

2014-03-11 Thread Timothy St. Clair (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1031?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930678#comment-13930678 ] Timothy St. Clair edited comment on MESOS-1031 at 3/11/14 6:17 PM:

[jira] [Commented] (MESOS-1079) Mesos python egg build failure on OS X Mavericks (Xcode 5.1)

2014-03-11 Thread Till Toenshoff (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1079?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930707#comment-13930707 ] Till Toenshoff commented on MESOS-1079: --- Some more information on this subject.

Re: Review Request 19009: Added an in-memory Storage implementation.

2014-03-11 Thread Ben Mahler
On March 11, 2014, 4:43 p.m., Dominic Hamon wrote: src/state/in_memory.cpp, line 65 https://reviews.apache.org/r/19009/diff/1/?file=515967#file515967line65 why convert from hashset to vector? As opposed to? The Storage interface is defined to return a vector for names() (see

Re: Review Request 19009: Added an in-memory Storage implementation.

2014-03-11 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19009/ --- (Updated March 11, 2014, 6:28 p.m.) Review request for mesos, Benjamin Hindman

Re: Review Request 19009: Added an in-memory Storage implementation.

2014-03-11 Thread Ben Mahler
On March 11, 2014, 4:51 p.m., Dominic Hamon wrote: src/state/in_memory.cpp, line 67 https://reviews.apache.org/r/19009/diff/1/?file=515967#file515967line67 s/values/keys/ :) - Ben --- This is an automatically generated

[jira] [Commented] (MESOS-1073) Upgrade gmock from 1.6 to 1.7 and set C++11 support

2014-03-11 Thread Timothy St. Clair (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1073?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930757#comment-13930757 ] Timothy St. Clair commented on MESOS-1073: -- Could you please post a link re:

Re: Review Request 18718: Added simple metrics API.

2014-03-11 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18718/ --- (Updated March 11, 2014, 11:46 a.m.) Review request for mesos, Benjamin

[jira] [Commented] (MESOS-1073) Upgrade gmock from 1.6 to 1.7 and set C++11 support

2014-03-11 Thread Dominic Hamon (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1073?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930760#comment-13930760 ] Dominic Hamon commented on MESOS-1073: -- Here are the release notes:

[jira] [Created] (MESOS-1080) cpplint.py doesn't analyze hpp files

2014-03-11 Thread Adam B (JIRA)
Adam B created MESOS-1080: - Summary: cpplint.py doesn't analyze hpp files Key: MESOS-1080 URL: https://issues.apache.org/jira/browse/MESOS-1080 Project: Mesos Issue Type: Bug Reporter:

[jira] [Commented] (MESOS-1079) Mesos python egg build failure on OS X Mavericks (Xcode 5.1)

2014-03-11 Thread Till Toenshoff (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1079?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930773#comment-13930773 ] Till Toenshoff commented on MESOS-1079: --- The proper fix here would be that Apple

Re: Question on executors

2014-03-11 Thread Benjamin Mahler
If you're just loading a .jar then I suppose you are ok, but I'm not sure if this is a wise design for running arbitrary .jars. You'll be able to share the JVM overhead but you'll also be sharing the same JVM runtime, is that well supported? What happens if both your executor .jar and the task

Review Request 19060: Updated cpplint.py to analyze hpp files.

2014-03-11 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19060/ --- Review request for mesos, Bernd Mathiske, Ben Mahler, and Niklas Nielsen.

Re: Review Request 19060: Updated cpplint.py to analyze hpp files.

2014-03-11 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19060/#review36826 --- support/cpplint.py

Re: Review Request 19060: Updated cpplint.py to analyze hpp files.

2014-03-11 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19060/ --- (Updated March 11, 2014, 12:15 p.m.) Review request for mesos, Bernd Mathiske,

[jira] [Commented] (MESOS-1080) cpplint.py doesn't analyze hpp files

2014-03-11 Thread Adam B (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1080?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930820#comment-13930820 ] Adam B commented on MESOS-1080: --- Up for review: https://reviews.apache.org/r/19060/

Re: Review Request 19060: Updated cpplint.py to analyze hpp files.

2014-03-11 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19060/ --- (Updated March 11, 2014, 12:24 p.m.) Review request for mesos, Bernd Mathiske,

Re: Review Request 19060: Updated cpplint.py to analyze hpp files.

2014-03-11 Thread Adam B
On March 11, 2014, 12:05 p.m., Dominic Hamon wrote: support/cpplint.py, line 3573 https://reviews.apache.org/r/19060/diff/1/?file=516611#file516611line3573 we should also start putting test sources alongside the sources they test if we want this method to do anything useful.

Re: Review Request 19060: Updated cpplint.py to analyze hpp files.

2014-03-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19060/#review36830 --- Patch looks great! Reviews applied: [19060] All tests passed. -

Re: Review Request 19060: Updated cpplint.py to analyze hpp files.

2014-03-11 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19060/#review36835 --- I also did chmod a+x for support/mesos-style.py, and it showed up

Re: Review Request 18718: Added simple metrics API.

2014-03-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18718/#review36837 --- Patch looks great! Reviews applied: [18718] All tests passed. -

Re: Review Request 18718: Added simple metrics API.

2014-03-11 Thread Dominic Hamon
On March 6, 2014, 4:20 p.m., Benjamin Hindman wrote: Before I dive too deep into this review, can you give an overview of your proposed next steps? In particular: (1) What's the concurrency story here? It appears as though multiple actors adding or removing metrics could causing

Re: Review Request 18823: Fixed MESOS-1055: explicit single-argument constructors

2014-03-11 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18823/#review36840 --- Ship it! Looks good to me include/mesos/resources.hpp

Review Request 19070: MESOS-471: added all command line switches to documentation.md

2014-03-11 Thread Jörn Franke
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19070/ --- Review request for mesos. Bugs: MESOS-471

Review Request 19068: Fix for build failure of Python-Egg on OSX clang 3.4

2014-03-11 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19068/ --- Review request for mesos, Adam B, Benjamin Hindman, Ben Mahler, Niklas Nielsen,

[jira] [Commented] (MESOS-1079) Mesos python egg build failure on OS X Mavericks (Xcode 5.1)

2014-03-11 Thread Till Toenshoff (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1079?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13930971#comment-13930971 ] Till Toenshoff commented on MESOS-1079: --- https://reviews.apache.org/r/19068/ Mesos

Re: Review Request 18386: Option reference cleanup in mesos.

2014-03-11 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18386/#review36842 --- src/slave/containerizer/mesos_containerizer.cpp

Re: Review Request 18966: change the datatype of cpushare from size_t to uint64_t because The actual type of size_t is platform-dependent .

2014-03-11 Thread Archana kumari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18966/ --- (Updated March 11, 2014, 9:36 p.m.) Review request for mesos. Repository:

Re: Review Request 19068: Fix for build failure of Python-Egg on OSX clang 3.4

2014-03-11 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19068/ --- (Updated March 11, 2014, 9:38 p.m.) Review request for mesos, Adam B, Benjamin

Re: Review Request 19009: Added an in-memory Storage implementation.

2014-03-11 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19009/#review36845 --- Ship it! src/state/in_memory.hpp

Re: Review Request 19068: Fix for build failure of Python-Egg on OSX clang 3.4

2014-03-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19068/#review36847 --- Patch looks great! Reviews applied: [19068] All tests passed. -

Re: Review Request 18311: Adds await on a tuple of futures.

2014-03-11 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18311/#review36849 --- 3rdparty/libprocess/include/process/collect.hpp

Re: Review Request 19008: Moved LevelDBStorageProcess and ZooKeeperStorageProcess out of the header files.

2014-03-11 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19008/#review36851 --- Ship it! Ship It! - Benjamin Hindman On March 11, 2014, 3:44

Re: Review Request 18966: change the datatype of cpushare from size_t to uint64_t because The actual type of size_t is platform-dependent .

2014-03-11 Thread Ben Mahler
On March 11, 2014, 6:07 p.m., Ben Mahler wrote: src/slave/containerizer/isolators/cgroups/cpushare.cpp, lines 60-61 https://reviews.apache.org/r/18966/diff/1/?file=515048#file515048line60 Please be sure to update the places where these are used as well as the argument type in

[jira] [Created] (MESOS-1081) Master should not deactivate authenticated framework/slave on new AuthenticateMessage unless new authentication succeeds.

2014-03-11 Thread Adam B (JIRA)
Adam B created MESOS-1081: - Summary: Master should not deactivate authenticated framework/slave on new AuthenticateMessage unless new authentication succeeds. Key: MESOS-1081 URL:

Review Request 19071: Fixed a TODO(bmahler): Consider namespacing stat for each subsystem (e.g. cgroups::memory::stat and cgroups::cpuacct::stat). in linux/cgroups.hpp

2014-03-11 Thread Archana kumari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19071/ --- Review request for mesos. Repository: mesos-git Description --- Fixed

Re: Review Request 19071: Fixed a TODO(bmahler): Consider namespacing stat for each subsystem (e.g. cgroups::memory::stat and cgroups::cpuacct::stat). in linux/cgroups.hpp

2014-03-11 Thread Archana kumari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19071/ --- (Updated March 11, 2014, 10:19 p.m.) Review request for mesos. Repository:

Re: Review Request 19071: Fixed a TODO(bmahler): Consider namespacing stat for each subsystem (e.g. cgroups::memory::stat and cgroups::cpuacct::stat). in linux/cgroups.hpp

2014-03-11 Thread Archana kumari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19071/ --- (Updated March 11, 2014, 10:19 p.m.) Review request for mesos. Repository:

Re: Review Request 18386: Option reference cleanup in mesos.

2014-03-11 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18386/ --- (Updated March 11, 2014, 3:20 p.m.) Review request for mesos and Ben Mahler.

Re: Review Request 18966: change the datatype of cpushare from size_t to uint64_t because The actual type of size_t is platform-dependent .

2014-03-11 Thread Archana kumari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18966/ --- (Updated March 11, 2014, 10:28 p.m.) Review request for mesos. Repository:

Re: Review Request 18966: change the datatype of cpushare from size_t to uint64_t because The actual type of size_t is platform-dependent .

2014-03-11 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18966/#review36857 --- src/linux/cgroups.hpp

Re: Review Request 18311: Adds await on a tuple of futures.

2014-03-11 Thread Till Toenshoff
On March 11, 2014, 1:26 a.m., Benjamin Hindman wrote: 3rdparty/libprocess/include/process/collect.hpp, lines 43-48 https://reviews.apache.org/r/18311/diff/5/?file=506398#file506398line43 After the inclusion of Future::after I'm not sure that this will be as useful since if you've

Re: Review Request 18311: Adds await on a tuple of futures.

2014-03-11 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18311/ --- (Updated March 11, 2014, 10:35 p.m.) Review request for mesos and Benjamin

Review Request 19072: Added tests for InMemoryStorage.

2014-03-11 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19072/ --- Review request for mesos, Benjamin Hindman and Vinod Kone. Bugs: MESOS-764

Re: Review Request 18311: Adds await on a tuple of futures.

2014-03-11 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18311/ --- (Updated March 11, 2014, 10:36 p.m.) Review request for mesos and Benjamin

Review Request 19073: Fixed a TODO(bmahler): Consider namespacing stat for each subsystem (e.g. cgroups::memory::stat and cgroups::cpuacct::stat).by writing wrappers for stat in in each namespace.

2014-03-11 Thread Archana kumari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19073/ --- Review request for mesos. Repository: mesos-git Description --- Fixed a

Review Request 19077: Only darwin/clang needs GTEST to supply its own tr1 tuple

2014-03-11 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19077/ --- Review request for mesos, Till Toenshoff and Vinod Kone. Bugs: MESOS-750

Re: Review Request 18966: change the datatype of cpushare from size_t to uint64_t because The actual type of size_t is platform-dependent .

2014-03-11 Thread Archana kumari
On March 11, 2014, 10:29 p.m., Dominic Hamon wrote: src/linux/cgroups.hpp, line 405 https://reviews.apache.org/r/18966/diff/3/?file=516748#file516748line405 there's almost certainly a cgroups.cpp change missing here. what kind of change are you hinting towards - Archana

Re: Review Request 18966: change the datatype of cpushare from size_t to uint64_t because The actual type of size_t is platform-dependent .

2014-03-11 Thread Archana kumari
On March 11, 2014, 10:29 p.m., Dominic Hamon wrote: src/linux/cgroups.hpp, line 405 https://reviews.apache.org/r/18966/diff/3/?file=516748#file516748line405 there's almost certainly a cgroups.cpp change missing here. Archana kumari wrote: what kind of change are you hinting

Re: Review Request 19077: Only darwin/clang needs GTEST to supply its own tr1 tuple

2014-03-11 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19077/ --- (Updated March 11, 2014, 4 p.m.) Review request for mesos, Till Toenshoff and

Re: Review Request 18946: Moved JNI code to separate library

2014-03-11 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18946/ --- (Updated March 12, 2014, 12:56 a.m.) Review request for mesos and Vinod Kone.

Re: building meso-0.17.0 failing on maverick

2014-03-11 Thread haja gecko
hi vinod/till i am not setting any aliases for this option. i thought i was using clang, but i may be wrong, in which case the os must be implicitly picking up the gcc-4.8 that i manually installed on mavericks. i went through several iterations of trying to build, so i may have changed stuff on

Re: Review Request 18966: change the datatype of cpushare from size_t to uint64_t because The actual type of size_t is platform-dependent .

2014-03-11 Thread Dominic Hamon
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18966/#review36869 --- Ship it! Ship It! - Dominic Hamon On March 11, 2014, 4:05 p.m.,

Re: Review Request 18946: Moved JNI code to separate library

2014-03-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18946/#review36894 --- Patch looks great! Reviews applied: [18946] All tests passed. -

Re: Review Request 18381: Added authentication support for slaves.

2014-03-11 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18381/ --- (Updated March 11, 2014, 6:51 p.m.) Review request for mesos and Vinod Kone.

Re: Review Request 18381: Added authentication support for slaves.

2014-03-11 Thread Adam B
On March 6, 2014, 4:07 p.m., Vinod Kone wrote: src/master/master.cpp, lines 2485-2486 https://reviews.apache.org/r/18381/diff/2/?file=511057#file511057line2485 Good point. If someone can spoof the pid in the authenticate message then it might deactivate a legitimate framework.

[jira] [Updated] (MESOS-1083) Error in CgroupsTest::SetUpTestCase() and TearDownTestCase()

2014-03-11 Thread Yan Xu (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-1083?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Yan Xu updated MESOS-1083: -- Description: The

Re: Review Request 18910: Removal of python-boto from the repo and add configure checks.

2014-03-11 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18910/#review36882 --- Looks good. I wonder if a more explicit dependency documentation

Review Request 19085: Implemented optional strictness in the Registrar.

2014-03-11 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19085/ --- Review request for mesos, Benjamin Hindman and Vinod Kone. Bugs: MESOS-764

  1   2   >