Review Request 38535: Fix ExamplesTest errors when user is root.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38535/ --- Review request for mesos, Jie Yu, Joris Van Remoortere, and Michael Park. Bugs:

Re: Review Request 38535: Fix ExamplesTest errors when user is root.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38535/#review99698 --- Patch looks great! Reviews applied: [38535] All tests passed. - M

Re: Review Request 38535: Fix ExamplesTest errors when user is root.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38535/ --- (Updated Sept. 20, 2015, 4:08 p.m.) Review request for mesos, Jie Yu, Joris Van

Re: Review Request 38535: Fix ExamplesTest errors when user is root.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38535/#review99704 --- Ship it! Thanks for the catch!! LGTM module the memory leak. src/

Re: Review Request 38535: Fix ExamplesTest errors when user is root.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38535/#review99705 --- Patch looks great! Reviews applied: [38535] All tests passed. - M

Re: Review Request 38535: Fix ExamplesTest errors when user is root.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38535/ --- (Updated Sept. 20, 2015, 5:04 p.m.) Review request for mesos, Jie Yu, Joris Van

Re: Review Request 38535: Fix ExamplesTest errors when user is root.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38535/#review99708 --- Patch looks great! Reviews applied: [38535] All tests passed. - M

Re: Review Request 38475: Maintenance Primitives: Add test for inverse offer filters.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38475/#review99687 --- Ship it! src/tests/master_maintenance_tests.cpp (lines 1230 - 1231

Re: Review Request 37873: Add quiesce logic in allocator

> On Sept. 17, 2015, 9:42 p.m., Vinod Kone wrote: > > src/tests/mesos.hpp, line 1608 > > > > > > Needs a test for the scheduler driver. I've only see one for the http > > scheduler library. > > > > Also, c

Re: Review Request 38514: Update QUIESCE to SUPPRESS in Mesos Call

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38514/#review99710 --- src/master/http.cpp (line 486)

Re: Review Request 37585: Maintenance primitives: Add a user doc.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37585/#review99711 --- Ship it! fixed inline. docs/maintenance.md (lines 139 - 140)

Re: Review Request 38473: Add flag to disable hostname lookup.

> On Sept. 19, 2015, 6:57 a.m., Cong Wang wrote: > > Why not just set --host_name=$LIBPROCESS_IP for your case since you anyway > > need to provide a flag? > > Guangya Liu wrote: > I also have the same question with Cong, @Marco, can you please show more > detail for why not using the solu

Re: Review Request 38516: Changed quiesceOffers to SuppressOffers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38516/ --- (Updated 九月 21, 2015, 12:11 a.m.) Review request for mesos and Vinod Kone. Ch

Re: Review Request 38514: Update QUIESCE to SUPPRESS in Mesos Call

> On 九月 20, 2015, 6:46 p.m., Vinod Kone wrote: > > src/master/http.cpp, line 486 > > > > > > this method should be called suppress too. > > > > > > to avoid conflict with the suppress marco, can you try

Re: Review Request 38514: Update QUIESCE to SUPPRESS in Mesos Call

> On Sept. 20, 2015, 6:46 p.m., Vinod Kone wrote: > > src/master/http.cpp, line 486 > > > > > > this method should be called suppress too. > > > > > > to avoid conflict with the suppress marco, can you

Re: Review Request 38516: Changed quiesceOffers to SuppressOffers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38516/#review99717 --- Ship it! Ship It! - Vinod Kone On Sept. 21, 2015, 12:11 a.m., Gu

Re: Review Request 38519: Change function quiesce() to suppressRes()

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38519/#review99718 --- src/master/http.cpp (line 486)

Review Request 38538: [VIA HAOSDENT] CMake: Add `CMAKE_NOOP` to common definitions file.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38538/ --- Review request for mesos, haosdent huang, Artem Harutyunyan, Joris Van Remoorter

Re: Review Request 38538: [VIA HAOSDENT] CMake: Add `CMAKE_NOOP` to common definitions file.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38538/#review99720 --- Ship it! Ship It! - haosdent huang On Sept. 21, 2015, 2:33 a.m.,

Review Request 38540: [VIA HAOSDENT] [2/2]Generate make batch file to build project in windows.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38540/ --- Review request for mesos, haosdent huang, Artem Harutyunyan, Joris Van Remoorter

Review Request 38539: [VIA HAOSDENT] [1/2]Add CMake macro VsBuildCommand in libprocess.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38539/ --- Review request for mesos, haosdent huang, Artem Harutyunyan, Joris Van Remoorter

Re: Review Request 38457: CMake: Fix MESOS-3250, a dynamic load error in Stout tests on OS X.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38457/ --- (Updated Sept. 21, 2015, 2:43 a.m.) Review request for mesos, Artem Harutyunyan

Review Request 38541: CMake: Add `Versions.cmake` as an analog to `versions.am`.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38541/ --- Review request for mesos, Artem Harutyunyan, Joris Van Remoortere, and Joseph Wu

Review Request 38542: CMake: Use version info from `Versions.cmake` instead of magic strings.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38542/ --- Review request for mesos, Artem Harutyunyan, Joris Van Remoortere, and Joseph Wu

Re: Review Request 38540: [VIA HAOSDENT] [2/2]Generate make batch file to build project in windows.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38540/#review99721 --- Patch looks great! Reviews applied: [38456, 38457, 38529, 38530, 38

Re: Review Request 38542: CMake: Use version info from `Versions.cmake` instead of magic strings.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38542/#review99722 --- Patch looks great! Reviews applied: [38456, 38457, 38529, 38530, 38

Re: Review Request 37993: WIP: Add explanatory comments for Allocator interface

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37993/ --- (Updated 九月 21, 2015, 5:02 a.m.) Review request for mesos, Adam B, Alexander Ru

Re: Review Request 37993: WIP: Add explanatory comments for Allocator interface

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37993/#review99723 --- Bad patch! Reviews applied: [37993] Failed command: ./support/appl

Re: Review Request 38514: Update QUIESCE to SUPPRESS in Mesos Call

> On 九月 20, 2015, 6:46 p.m., Vinod Kone wrote: > > src/master/http.cpp, line 486 > > > > > > this method should be called suppress too. > > > > > > to avoid conflict with the suppress marco, can you try

Re: Review Request 38516: Changed quiesceOffers to SuppressOffers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38516/ --- (Updated 九月 21, 2015, 6:05 a.m.) Review request for mesos and Vinod Kone. Bug

Re: Review Request 38514: Update QUIESCE to SUPPRESS in Mesos Call

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38514/ --- (Updated 九月 21, 2015, 6:05 a.m.) Review request for mesos and Vinod Kone. Bug

Review Request 38544: Update macro of suppress to SUPPRESS

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38544/ --- Review request for mesos. Repository: mesos Description --- The macro su

Re: Review Request 38519: Change function quiesce() to suppress()

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38519/ --- (Updated 九月 21, 2015, 6:06 a.m.) Review request for mesos and Vinod Kone. Sum

Re: Review Request 37993: WIP: Add explanatory comments for Allocator interface

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37993/ --- (Updated 九月 21, 2015, 6:20 a.m.) Review request for mesos, Adam B, Alexander Ru

Re: Review Request 38514: Update QUIESCE to SUPPRESS in Mesos Call

> On 九月 20, 2015, 6:46 p.m., Vinod Kone wrote: > > src/master/http.cpp, line 486 > > > > > > this method should be called suppress too. > > > > > > to avoid conflict with the suppress marco, can you try

Re: Review Request 38544: libprocess: Updated macro of suppress to SUPPRESS

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38544/ --- (Updated 九月 21, 2015, 6:22 a.m.) Review request for mesos. Summary (updated)

Re: Review Request 38051: Only update the task status when its old status is not terminal.

> On Sept. 17, 2015, 10:25 p.m., Vinod Kone wrote: > > Can you write a test for this? > > Yong Qiao Wang wrote: > I find the code changes in this patch does not be tested with an > end-to-end case except to check the error log messages of master, so my test > strategy are: > > 1.