You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mesos.apache.org by Zuyu Zhang <hi...@gmail.com> on 2014/06/26 20:55:08 UTC
Re: Review Request 22539: Add the low level scheduler example using
libprocess
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22539/
-----------------------------------------------------------
(Updated June 26, 2014, 6:55 p.m.)
Review request for mesos, Benjamin Hindman and Ben Mahler.
Summary (updated)
-----------------
Add the low level scheduler example using libprocess
Repository: mesos-git
Description (updated)
-------
Add the low level scheduler example using libprocess
Diffs (updated)
-----
src/Makefile.am 3c8b91a2ef4f90137bc7f049c31295bb198ef766
src/examples/low_level_scheduler_libprocess.cpp PRE-CREATION
src/tests/examples_tests.cpp 34f0233aca3433faba7528ac8c354100b8d3a4f7
src/tests/low_level_scheduler_libprocess_test.sh PRE-CREATION
Diff: https://reviews.apache.org/r/22539/diff/
Testing (updated)
-------
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from ExamplesTest
[ RUN ] ExamplesTest.LowLevelSchedulerLibprocess
[ OK ] ExamplesTest.LowLevelSchedulerLibprocess (1670 ms)
[----------] 1 test from ExamplesTest (1670 ms total)
[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (1684 ms total)
[ PASSED ] 1 test.
Thanks,
Zuyu Zhang
Re: Review Request 22539: Add the low level scheduler example using
libprocess
Posted by Zuyu Zhang <hi...@gmail.com>.
> On June 30, 2014, 10:08 p.m., Vinod Kone wrote:
> > src/Makefile.am, line 955
> > <https://reviews.apache.org/r/22539/diff/9/?file=619384#file619384line955>
> >
> > All our example frameworks use libprocess. Did you add the "libprocess" suffix because you are planning to write other example frameworks that don't use libprocess? if not, kill that suffix.
There will be a pthread version with a new review thread.
> On June 30, 2014, 10:08 p.m., Vinod Kone wrote:
> > src/tests/examples_tests.cpp, line 27
> > <https://reviews.apache.org/r/22539/diff/9/?file=619386#file619386line27>
> >
> > s/Libprocess//
Reserved for another pthread version.
> On June 30, 2014, 10:08 p.m., Vinod Kone wrote:
> > src/tests/examples_tests.cpp, line 28
> > <https://reviews.apache.org/r/22539/diff/9/?file=619386#file619386line28>
> >
> > s/libprocess//
See above.
> On June 30, 2014, 10:08 p.m., Vinod Kone wrote:
> > src/examples/low_level_scheduler_libprocess.cpp, line 328
> > <https://reviews.apache.org/r/22539/diff/9/?file=619385#file619385line328>
> >
> > Can this be const?
No, because we need a const version of scheduler::Mesos::send.
- Zuyu
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22539/#review47018
-----------------------------------------------------------
On June 27, 2014, 12:47 p.m., Zuyu Zhang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/22539/
> -----------------------------------------------------------
>
> (Updated June 27, 2014, 12:47 p.m.)
>
>
> Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
>
>
> Repository: mesos-git
>
>
> Description
> -------
>
> Add the low level scheduler example using libprocess
>
>
> Diffs
> -----
>
> src/Makefile.am 902792782e760d5e0a73fb7aa72aef003c0af9c0
> src/examples/low_level_scheduler_libprocess.cpp PRE-CREATION
> src/tests/examples_tests.cpp 34f0233aca3433faba7528ac8c354100b8d3a4f7
> src/tests/low_level_scheduler_libprocess_test.sh PRE-CREATION
>
> Diff: https://reviews.apache.org/r/22539/diff/
>
>
> Testing
> -------
>
> [==========] Running 1 test from 1 test case.
> [----------] Global test environment set-up.
> [----------] 1 test from ExamplesTest
> [ RUN ] ExamplesTest.LowLevelSchedulerLibprocess
> [ OK ] ExamplesTest.LowLevelSchedulerLibprocess (1670 ms)
> [----------] 1 test from ExamplesTest (1670 ms total)
>
> [----------] Global test environment tear-down
> [==========] 1 test from 1 test case ran. (1684 ms total)
> [ PASSED ] 1 test.
>
>
> Thanks,
>
> Zuyu Zhang
>
>
Re: Review Request 22539: Add the low level scheduler example using
libprocess
Posted by Vinod Kone <vi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22539/#review47018
-----------------------------------------------------------
looks good. some minor fixes before we can commit this.
src/Makefile.am
<https://reviews.apache.org/r/22539/#comment82614>
All our example frameworks use libprocess. Did you add the "libprocess" suffix because you are planning to write other example frameworks that don't use libprocess? if not, kill that suffix.
src/examples/low_level_scheduler_libprocess.cpp
<https://reviews.apache.org/r/22539/#comment82616>
s/to/with/
src/examples/low_level_scheduler_libprocess.cpp
<https://reviews.apache.org/r/22539/#comment82617>
How about:
TODO(zuyu): Do batch processing of UPDATE events.
src/examples/low_level_scheduler_libprocess.cpp
<https://reviews.apache.org/r/22539/#comment82618>
s/failed/terminated/
src/examples/low_level_scheduler_libprocess.cpp
<https://reviews.apache.org/r/22539/#comment82619>
s/failed/terminated/
src/examples/low_level_scheduler_libprocess.cpp
<https://reviews.apache.org/r/22539/#comment82620>
This should be CHECK because this is not possible.
src/examples/low_level_scheduler_libprocess.cpp
<https://reviews.apache.org/r/22539/#comment82621>
Describe what you are printing.
Also include Offer id.
src/examples/low_level_scheduler_libprocess.cpp
<https://reviews.apache.org/r/22539/#comment82622>
s/task_info/taskInfo/
src/examples/low_level_scheduler_libprocess.cpp
<https://reviews.apache.org/r/22539/#comment82623>
end the comment with period.
src/examples/low_level_scheduler_libprocess.cpp
<https://reviews.apache.org/r/22539/#comment82615>
Can this be const?
src/tests/examples_tests.cpp
<https://reviews.apache.org/r/22539/#comment82624>
s/Libprocess//
src/tests/examples_tests.cpp
<https://reviews.apache.org/r/22539/#comment82625>
s/libprocess//
- Vinod Kone
On June 27, 2014, 12:47 p.m., Zuyu Zhang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/22539/
> -----------------------------------------------------------
>
> (Updated June 27, 2014, 12:47 p.m.)
>
>
> Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
>
>
> Repository: mesos-git
>
>
> Description
> -------
>
> Add the low level scheduler example using libprocess
>
>
> Diffs
> -----
>
> src/Makefile.am 902792782e760d5e0a73fb7aa72aef003c0af9c0
> src/examples/low_level_scheduler_libprocess.cpp PRE-CREATION
> src/tests/examples_tests.cpp 34f0233aca3433faba7528ac8c354100b8d3a4f7
> src/tests/low_level_scheduler_libprocess_test.sh PRE-CREATION
>
> Diff: https://reviews.apache.org/r/22539/diff/
>
>
> Testing
> -------
>
> [==========] Running 1 test from 1 test case.
> [----------] Global test environment set-up.
> [----------] 1 test from ExamplesTest
> [ RUN ] ExamplesTest.LowLevelSchedulerLibprocess
> [ OK ] ExamplesTest.LowLevelSchedulerLibprocess (1670 ms)
> [----------] 1 test from ExamplesTest (1670 ms total)
>
> [----------] Global test environment tear-down
> [==========] 1 test from 1 test case ran. (1684 ms total)
> [ PASSED ] 1 test.
>
>
> Thanks,
>
> Zuyu Zhang
>
>
Re: Review Request 22539: Add the low level scheduler example using
libprocess
Posted by Mesos ReviewBot <de...@mesos.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22539/#review46936
-----------------------------------------------------------
Patch looks great!
Reviews applied: [22539]
All tests passed.
- Mesos ReviewBot
On June 27, 2014, 12:47 p.m., Zuyu Zhang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/22539/
> -----------------------------------------------------------
>
> (Updated June 27, 2014, 12:47 p.m.)
>
>
> Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
>
>
> Repository: mesos-git
>
>
> Description
> -------
>
> Add the low level scheduler example using libprocess
>
>
> Diffs
> -----
>
> src/Makefile.am 902792782e760d5e0a73fb7aa72aef003c0af9c0
> src/examples/low_level_scheduler_libprocess.cpp PRE-CREATION
> src/tests/examples_tests.cpp 34f0233aca3433faba7528ac8c354100b8d3a4f7
> src/tests/low_level_scheduler_libprocess_test.sh PRE-CREATION
>
> Diff: https://reviews.apache.org/r/22539/diff/
>
>
> Testing
> -------
>
> [==========] Running 1 test from 1 test case.
> [----------] Global test environment set-up.
> [----------] 1 test from ExamplesTest
> [ RUN ] ExamplesTest.LowLevelSchedulerLibprocess
> [ OK ] ExamplesTest.LowLevelSchedulerLibprocess (1670 ms)
> [----------] 1 test from ExamplesTest (1670 ms total)
>
> [----------] Global test environment tear-down
> [==========] 1 test from 1 test case ran. (1684 ms total)
> [ PASSED ] 1 test.
>
>
> Thanks,
>
> Zuyu Zhang
>
>
Re: Review Request 22539: Add the low level scheduler example using
libprocess
Posted by Vinod Kone <vi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22539/#review47116
-----------------------------------------------------------
Ship it!
Ship It!
- Vinod Kone
On July 1, 2014, 12:58 a.m., Zuyu Zhang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/22539/
> -----------------------------------------------------------
>
> (Updated July 1, 2014, 12:58 a.m.)
>
>
> Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
>
>
> Repository: mesos-git
>
>
> Description
> -------
>
> Add the low level scheduler example using libprocess
>
>
> Diffs
> -----
>
> src/Makefile.am 918b0d04a5de69a9213e3d31c8f9424756e4ade5
> src/examples/low_level_scheduler_libprocess.cpp PRE-CREATION
> src/tests/examples_tests.cpp 34f0233aca3433faba7528ac8c354100b8d3a4f7
> src/tests/low_level_scheduler_libprocess_test.sh PRE-CREATION
>
> Diff: https://reviews.apache.org/r/22539/diff/
>
>
> Testing
> -------
>
> [==========] Running 1 test from 1 test case.
> [----------] Global test environment set-up.
> [----------] 1 test from ExamplesTest
> [ RUN ] ExamplesTest.LowLevelSchedulerLibprocess
> [ OK ] ExamplesTest.LowLevelSchedulerLibprocess (1670 ms)
> [----------] 1 test from ExamplesTest (1670 ms total)
>
> [----------] Global test environment tear-down
> [==========] 1 test from 1 test case ran. (1684 ms total)
> [ PASSED ] 1 test.
>
>
> Thanks,
>
> Zuyu Zhang
>
>
Re: Review Request 22539: Add the low level scheduler example using
libprocess
Posted by Mesos ReviewBot <de...@mesos.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22539/#review47036
-----------------------------------------------------------
Patch looks great!
Reviews applied: [22539]
All tests passed.
- Mesos ReviewBot
On July 1, 2014, 12:58 a.m., Zuyu Zhang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/22539/
> -----------------------------------------------------------
>
> (Updated July 1, 2014, 12:58 a.m.)
>
>
> Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
>
>
> Repository: mesos-git
>
>
> Description
> -------
>
> Add the low level scheduler example using libprocess
>
>
> Diffs
> -----
>
> src/Makefile.am 918b0d04a5de69a9213e3d31c8f9424756e4ade5
> src/examples/low_level_scheduler_libprocess.cpp PRE-CREATION
> src/tests/examples_tests.cpp 34f0233aca3433faba7528ac8c354100b8d3a4f7
> src/tests/low_level_scheduler_libprocess_test.sh PRE-CREATION
>
> Diff: https://reviews.apache.org/r/22539/diff/
>
>
> Testing
> -------
>
> [==========] Running 1 test from 1 test case.
> [----------] Global test environment set-up.
> [----------] 1 test from ExamplesTest
> [ RUN ] ExamplesTest.LowLevelSchedulerLibprocess
> [ OK ] ExamplesTest.LowLevelSchedulerLibprocess (1670 ms)
> [----------] 1 test from ExamplesTest (1670 ms total)
>
> [----------] Global test environment tear-down
> [==========] 1 test from 1 test case ran. (1684 ms total)
> [ PASSED ] 1 test.
>
>
> Thanks,
>
> Zuyu Zhang
>
>
Re: Review Request 22539: Add the low level scheduler example using
libprocess
Posted by Zuyu Zhang <hi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22539/
-----------------------------------------------------------
(Updated July 1, 2014, 12:58 a.m.)
Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
Repository: mesos-git
Description
-------
Add the low level scheduler example using libprocess
Diffs (updated)
-----
src/Makefile.am 918b0d04a5de69a9213e3d31c8f9424756e4ade5
src/examples/low_level_scheduler_libprocess.cpp PRE-CREATION
src/tests/examples_tests.cpp 34f0233aca3433faba7528ac8c354100b8d3a4f7
src/tests/low_level_scheduler_libprocess_test.sh PRE-CREATION
Diff: https://reviews.apache.org/r/22539/diff/
Testing
-------
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from ExamplesTest
[ RUN ] ExamplesTest.LowLevelSchedulerLibprocess
[ OK ] ExamplesTest.LowLevelSchedulerLibprocess (1670 ms)
[----------] 1 test from ExamplesTest (1670 ms total)
[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (1684 ms total)
[ PASSED ] 1 test.
Thanks,
Zuyu Zhang
Re: Review Request 22539: Add the low level scheduler example using
libprocess
Posted by Zuyu Zhang <hi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22539/
-----------------------------------------------------------
(Updated June 27, 2014, 12:47 p.m.)
Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
Repository: mesos-git
Description
-------
Add the low level scheduler example using libprocess
Diffs
-----
src/Makefile.am 902792782e760d5e0a73fb7aa72aef003c0af9c0
src/examples/low_level_scheduler_libprocess.cpp PRE-CREATION
src/tests/examples_tests.cpp 34f0233aca3433faba7528ac8c354100b8d3a4f7
src/tests/low_level_scheduler_libprocess_test.sh PRE-CREATION
Diff: https://reviews.apache.org/r/22539/diff/
Testing
-------
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from ExamplesTest
[ RUN ] ExamplesTest.LowLevelSchedulerLibprocess
[ OK ] ExamplesTest.LowLevelSchedulerLibprocess (1670 ms)
[----------] 1 test from ExamplesTest (1670 ms total)
[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (1684 ms total)
[ PASSED ] 1 test.
Thanks,
Zuyu Zhang
Re: Review Request 22539: Add the low level scheduler example using
libprocess
Posted by Zuyu Zhang <hi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22539/
-----------------------------------------------------------
(Updated June 27, 2014, 12:47 p.m.)
Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
Repository: mesos-git
Description
-------
Add the low level scheduler example using libprocess
Diffs
-----
src/Makefile.am 902792782e760d5e0a73fb7aa72aef003c0af9c0
src/examples/low_level_scheduler_libprocess.cpp PRE-CREATION
src/tests/examples_tests.cpp 34f0233aca3433faba7528ac8c354100b8d3a4f7
src/tests/low_level_scheduler_libprocess_test.sh PRE-CREATION
Diff: https://reviews.apache.org/r/22539/diff/
Testing
-------
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from ExamplesTest
[ RUN ] ExamplesTest.LowLevelSchedulerLibprocess
[ OK ] ExamplesTest.LowLevelSchedulerLibprocess (1670 ms)
[----------] 1 test from ExamplesTest (1670 ms total)
[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (1684 ms total)
[ PASSED ] 1 test.
Thanks,
Zuyu Zhang
Re: Review Request 22539: Add the low level scheduler example using
libprocess
Posted by Zuyu Zhang <hi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22539/
-----------------------------------------------------------
(Updated June 27, 2014, 12:46 p.m.)
Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
Changes
-------
Fixed a typo in Makefile.am
Repository: mesos-git
Description
-------
Add the low level scheduler example using libprocess
Diffs (updated)
-----
src/Makefile.am 902792782e760d5e0a73fb7aa72aef003c0af9c0
src/examples/low_level_scheduler_libprocess.cpp PRE-CREATION
src/tests/examples_tests.cpp 34f0233aca3433faba7528ac8c354100b8d3a4f7
src/tests/low_level_scheduler_libprocess_test.sh PRE-CREATION
Diff: https://reviews.apache.org/r/22539/diff/
Testing
-------
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from ExamplesTest
[ RUN ] ExamplesTest.LowLevelSchedulerLibprocess
[ OK ] ExamplesTest.LowLevelSchedulerLibprocess (1670 ms)
[----------] 1 test from ExamplesTest (1670 ms total)
[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (1684 ms total)
[ PASSED ] 1 test.
Thanks,
Zuyu Zhang
Re: Review Request 22539: Add the low level scheduler example using
libprocess
Posted by Zuyu Zhang <hi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22539/
-----------------------------------------------------------
(Updated June 27, 2014, 12:51 a.m.)
Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
Repository: mesos-git
Description
-------
Add the low level scheduler example using libprocess
Diffs
-----
src/Makefile.am 902792782e760d5e0a73fb7aa72aef003c0af9c0
src/examples/low_level_scheduler_libprocess.cpp PRE-CREATION
src/tests/examples_tests.cpp 34f0233aca3433faba7528ac8c354100b8d3a4f7
src/tests/low_level_scheduler_libprocess_test.sh PRE-CREATION
Diff: https://reviews.apache.org/r/22539/diff/
Testing
-------
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from ExamplesTest
[ RUN ] ExamplesTest.LowLevelSchedulerLibprocess
[ OK ] ExamplesTest.LowLevelSchedulerLibprocess (1670 ms)
[----------] 1 test from ExamplesTest (1670 ms total)
[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (1684 ms total)
[ PASSED ] 1 test.
Thanks,
Zuyu Zhang
Re: Review Request 22539: Add the low level scheduler example using
libprocess
Posted by Zuyu Zhang <hi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22539/
-----------------------------------------------------------
(Updated June 27, 2014, 12:51 a.m.)
Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
Repository: mesos-git
Description
-------
Add the low level scheduler example using libprocess
Diffs (updated)
-----
src/Makefile.am 902792782e760d5e0a73fb7aa72aef003c0af9c0
src/examples/low_level_scheduler_libprocess.cpp PRE-CREATION
src/tests/examples_tests.cpp 34f0233aca3433faba7528ac8c354100b8d3a4f7
src/tests/low_level_scheduler_libprocess_test.sh PRE-CREATION
Diff: https://reviews.apache.org/r/22539/diff/
Testing
-------
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from ExamplesTest
[ RUN ] ExamplesTest.LowLevelSchedulerLibprocess
[ OK ] ExamplesTest.LowLevelSchedulerLibprocess (1670 ms)
[----------] 1 test from ExamplesTest (1670 ms total)
[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (1684 ms total)
[ PASSED ] 1 test.
Thanks,
Zuyu Zhang