-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36760/#review92855
-----------------------------------------------------------
Looks very good !
src/messages/messages.proto (line 352)
<https://reviews.apache.org/r/36760/#comment147108>
s/required/optional
src/slave/slave.cpp (line 1299)
<https://reviews.apache.org/r/36760/#comment147109>
Option<UPID frameworkPid;
The default constructor already initializes to None.
src/slave/slave.cpp (line 1753)
<https://reviews.apache.org/r/36760/#comment147110>
Curious, why not UPID() as is the case at other places ?
src/slave/slave.cpp (line 3030)
<https://reviews.apache.org/r/36760/#comment147113>
s/to/at
src/slave/state.hpp (line 253)
<https://reviews.apache.org/r/36760/#comment147111>
s/do not have a a/do not have a
- Anand Mazumdar
On July 24, 2015, 1:45 a.m., Ben Mahler wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/36760/
> -----------------------------------------------------------
>
> (Updated July 24, 2015, 1:45 a.m.)
>
>
> Review request for mesos, Anand Mazumdar and Vinod Kone.
>
>
> Bugs: MESOS-3132
> https://issues.apache.org/jira/browse/MESOS-3132
>
>
> Repository: mesos
>
>
> Description
> -------
>
> This is anticipation of HTTP scheduler support in 0.24.0.
> Note that the 'pid' is set for driver-based schedulers. The
> corresponding master changes to not set 'pid' for HTTP
> schedulers have not occurred yet.
>
>
> Diffs
> -----
>
> src/master/master.cpp 2f00f240ed2cd59ec0c2eae7fd2567f0edb8d9e0
> src/messages/messages.proto 165a16d91616b75c839c7fbf188dc49f029a8c45
> src/slave/slave.hpp dec4ca8323e151a6d0f9139214ff0ef6e3e3375a
> src/slave/slave.cpp dc12c45516ab39d74a5c29b657f22f74d0acf24e
> src/slave/state.hpp 4e00468a777145e3c61b8dee7dfe496f8d65b0e4
> src/tests/mesos.hpp 69134e1c2664ca24a1ecd80a662c841311104a6a
> src/tests/mesos.cpp f09ef0f99573716de8905f49dcc0c9df20e31ea9
> src/tests/slave_tests.cpp e1390ad84b0003052681600deb9ca518defc0970
>
> Diff: https://reviews.apache.org/r/36760/diff/
>
>
> Testing
> -------
>
> Added tests in [r/36761](https://reviews.apache.org/r/36761/).
>
>
> Thanks,
>
> Ben Mahler
>
>
|