mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alexander Rukletsov <ruklet...@gmail.com>
Subject Re: Review Request 65571: Handle 'None' passed from the MasterDetector in 'Master::detect()'.
Date Thu, 22 Mar 2018 14:11:00 GMT


> On March 22, 2018, 2:08 p.m., Alexander Rukletsov wrote:
> > Ship It!

Please use past simple in Summary for consistency.


- Alexander


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


On March 22, 2018, 12:06 p.m., Benno Evers wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65571/
> -----------------------------------------------------------
> 
> (Updated March 22, 2018, 12:06 p.m.)
> 
> 
> Review request for mesos, Andrei Budnik and Alexander Rukletsov.
> 
> 
> Bugs: MESOS-8550
>     https://issues.apache.org/jira/browse/MESOS-8550
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> The function `MasterDetector::detect()` returns a value of type
> `Future<Option<MasterInfo>>`, which, according to its documentation,
> can be `None` if an election occured and no master is elected.
> 
> However, the code in `Master::detected()` was only handling the
> cases of a failed future or a valid `MasterInfo` object.
> 
> *NOTE*: This commit does not add a corresponding unit test, since
> that would require starting a non-leading master. For the
> ZooKeeperMasterDetector, this is blocked by MESOS-2976, and an API
> change to make this possible with the StandaloneMasterDetector
> would add a lot of complexity to the `cluster::Master::start()`
> function for a feature that is unlikely to be re-used in any other
> test.
> 
> 
> Diffs
> -----
> 
>   src/master/master.cpp f0affa09dcf41b6a800e18105ad047bcffdfd3ec 
> 
> 
> Diff: https://reviews.apache.org/r/65571/diff/5/
> 
> 
> Testing
> -------
> 
> `./mesos-tests` (See also last paragraph in description)
> 
> 
> Thanks,
> 
> Benno Evers
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message