Re: Review Request 13757: Fixed an issue where the Master unnecessarily sends a "Framework failed over" message when the scheduler driver retries an initial failover re-registration.

[Vinod Kone]

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


Can you tag MESOS-488 instead?


src/master/master.cpp
<https://reviews.apache.org/r/13757/#comment49861>

    A comment here would be nice.
    
    Also, why not just ignore these messages in Master::reregisterFramework()?


- Vinod Kone


On Aug. 23, 2013, 1:15 a.m., Ben Mahler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13757/
> -----------------------------------------------------------
> 
> (Updated Aug. 23, 2013, 1:15 a.m.)
> 
> 
> Review request for mesos, Benjamin Hindman and Vinod Kone.
> 
> 
> Bugs: MESOS-659
>     https://issues.apache.org/jira/browse/MESOS-659
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> See MESOS-659.
> 
> 
> Diffs
> -----
> 
>   src/master/master.cpp d53b8bb97da45834790cca6e04b70b969a8d3453 
>   src/tests/fault_tolerance_tests.cpp 
> 10e52c401476eb8416361de49b8e4061bb7ac4f3 
> 
> Diff: https://reviews.apache.org/r/13757/diff/
> 
> 
> Testing
> -------
> 
> Added a test that catches this case when not fixed.
> 
> 
> Thanks,
> 
> Ben Mahler
> 
>