You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Guangya Liu <gy...@gmail.com> on 2015/09/02 16:48:29 UTC
Re: Review Request 37500: Update the FrameworkInfo.user on scheduler
failover
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37500/#review97456
-----------------------------------------------------------
src/slave/slave.cpp (line 2133)
<https://reviews.apache.org/r/37500/#comment153333>
This message needs to be updated as u are not only updating pid but also user name
- Guangya Liu
On 八月 25, 2015, 8:25 p.m., Aditi Dixit wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37500/
> -----------------------------------------------------------
>
> (Updated 八月 25, 2015, 8:25 p.m.)
>
>
> Review request for mesos and Vinod Kone.
>
>
> Bugs: MESOS-3240
> https://issues.apache.org/jira/browse/MESOS-3240
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Added user to master's state, slave's state (not exposed) and updated user in all slaves that have the registered framework. Added the test too.
>
>
> Diffs
> -----
>
> src/master/http.cpp 37d76ee72f6a037f551bf2609e9393e16b496e44
> src/master/master.hpp 36c67599ef2c470da8d95f2caf926a154342d2cc
> src/master/master.cpp 95207d24db0aa052eb70c4cc7eb75d0611c365cf
> src/messages/messages.proto 8977d8e0f3b16003128b6b9cab556a7b224f083c
> src/slave/slave.hpp 09172f7ed547049b3bd169b3db9be94e14f6bc39
> src/slave/slave.cpp 2a99abc00c525a93508d38e74d351d2f36572d86
> src/tests/fault_tolerance_tests.cpp 89cb18be96cd60fb77fbcc4acd08cebdcf1ba075
> src/tests/master_tests.cpp 8a6b98b9f59ead20f537eb60b5084feed069a5b1
> src/tests/slave_recovery_tests.cpp e1392a2235ff51dac7a3cb7cd3e8edf8406864fc
> src/tests/slave_tests.cpp d55e9dd4f4eb84a8fda85439e31a38e70890b377
>
> Diff: https://reviews.apache.org/r/37500/diff/
>
>
> Testing
> -------
>
> make check
>
>
> Thanks,
>
> Aditi Dixit
>
>