Re: Review Request 23187: Fixed a regression in ExecutorInfoChecker to allow an executor with the same id but different executor info as long as it is on a different slave.

[Jie Yu]

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

Ship it!


LGTM

- Jie Yu


On July 1, 2014, 12:21 a.m., Vinod Kone wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/23187/
> -----------------------------------------------------------
> 
> (Updated July 1, 2014, 12:21 a.m.)
> 
> 
> Review request for mesos, Benjamin Hindman and Jiang Yan Xu.
> 
> 
> Bugs: MESOS-1555
>     https://issues.apache.org/jira/browse/MESOS-1555
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> See summary and bug for details.
> 
> 
> Diffs
> -----
> 
>   src/master/master.cpp 474014b24982449f2f64c2e8d835c25a28cbcbb8 
>   src/tests/master_authorization_tests.cpp 
> 478041cdea533e548ca92c4b8e8c793554855969 
>   src/tests/resource_offers_tests.cpp 
> 3ec688abea1557a17526589b31f58ad6e53abec4 
> 
> Diff: https://reviews.apache.org/r/23187/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> make GTEST_FILTER="*PendingExecutorInfoDiffersOnDifferentSlaves*" 
> GTEST_REPEAT=1000 GTEST_BREAK_ON_FAILURE
> 
> 
> Thanks,
> 
> Vinod Kone
> 
>