-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46923/#review131489
-----------------------------------------------------------



You can put `MESOS-1575` to the `bugs` column at right.


src/master/master.cpp (line 7341)
<https://reviews.apache.org/r/46923/#comment195498>

    s/failoverTimeout_/failoverTimeout



src/master/master.cpp (line 7342)
<https://reviews.apache.org/r/46923/#comment195497>

    two spaces is enough


- Guangya Liu


On 五月 3, 2016, 3:31 a.m., Jose Guilherme Vanz wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46923/
> -----------------------------------------------------------
> 
> (Updated 五月 3, 2016, 3:31 a.m.)
> 
> 
> Review request for mesos, Kevin Sweeney and Vinod Kone.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Frameworks were allowed to subscribe themselves with invalid failover
> timeout. For this reason, a validation has been made in the master to
> deny framework subscription if it set a invalid value for the failover
> timeout. MESOS-1575
> 
> 
> Diffs
> -----
> 
>   src/master/master.hpp 3e55114ee7866e06513071e86e15608099dae052 
>   src/master/master.cpp 6d3e0f7c634690a35eec1ce827b705e04c3af87e 
>   src/tests/master_tests.cpp 8e00753fcbcd0cae1d08aad387b08aa17c7f63ad 
> 
> Diff: https://reviews.apache.org/r/46923/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jose Guilherme Vanz
> 
>

Reply via email to