> On May 10, 2016, 1:27 a.m., Cody Maloney wrote:
> > src/module/manager.cpp, line 377
> > <https://reviews.apache.org/r/47123/diff/3/?file=1376776#file1376776line377>
> >
> >     Is there a semantic difference between loading modules one at a time 
> > vs. just passing them all at once to loadManifest?
> >     
> >     I'm worried that the behavior here could differ slightly from the 
> > --modules flag which calls loadManifest with all the modules at once.

There is no semantic difference as of now. Even with the `--modules` flag, the 
modules are loaded one at a time, without any notion of interdependency. In 
future, if and when we add some other checks, say to find mutually exclusive 
modules, or interdependent modules, we might want to process the module 
manifests first (without actually loading the modules) and then load the 
modules at a later time. In that case, the change will be local to 
module/manager.cpp and so I wouldn't worry about it for now.


> On May 10, 2016, 1:27 a.m., Cody Maloney wrote:
> > src/module/manager.cpp, line 358
> > <https://reviews.apache.org/r/47123/diff/3/?file=1376776#file1376776line358>
> >
> >     Should document the sort order / load order based on filenames inside 
> > the doc changes to accompany this (I don't see it anywhere currently)

Added to the help message.


- Kapil


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


On May 19, 2016, 5:51 p.m., Kapil Arya wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47123/
> -----------------------------------------------------------
> 
> (Updated May 19, 2016, 5:51 p.m.)
> 
> 
> Review request for mesos, Cody Maloney and Till Toenshoff.
> 
> 
> Bugs: MESOS-5173
>     https://issues.apache.org/jira/browse/MESOS-5173
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> This allows the operator to use separate manifest JSON files for each
> module.  Previously, one had to merge all module manifest files into a
> single JSON file before passing on to the master/agent.
> 
> 
> Diffs
> -----
> 
>   src/master/flags.hpp d08509667c919e9cfd076762b622c18732edf910 
>   src/master/flags.cpp ceb4bd4f863d3857eede8287041a53d66f1c9e4c 
>   src/master/main.cpp d00bbef2c4fb74544056ab81aeb4fcd6625b89fa 
>   src/module/manager.hpp 9944af0daf6c9cb5a8ff338099401b1db88ee237 
>   src/module/manager.cpp 9f88ec3addab59e4a40b0b40612518178d535aa5 
>   src/sched/flags.hpp b4ca12b667283cee1f96a4b421fcf3b06bbe59d7 
>   src/sched/sched.cpp 9e55885704d5c4a8bc0e25e324b9c65e7bc34798 
>   src/slave/flags.hpp 3363412099ca2841e175bd6b6ed3d5f13605e6f0 
>   src/slave/flags.cpp 2d59329997197966f7d30d6f1375a988edbceb9c 
>   src/slave/main.cpp 13ddfb9ae5a1ca240b738e67391ce5c5fc9ac0b6 
>   src/tests/flags.hpp ae232b1a087edfaf678bd1c67bc509efd6c740d8 
>   src/tests/main.cpp efec92286c78f7783e1de0781eb9e61c202a4fb8 
> 
> Diff: https://reviews.apache.org/r/47123/diff/
> 
> 
> Testing
> -------
> 
> Manual testing with:
> 1. --module_dir
> 2. --modules
> 3. --module_dir in conjunction with --modules
> 
> The first two succeeded while the third failed as expected.
> 
> 
> Thanks,
> 
> Kapil Arya
> 
>

Reply via email to