----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44694/#review123119 -----------------------------------------------------------
Patch looks great! Reviews applied: [44694] Passed command: export OS='ubuntu:14.04' CONFIGURATION='--verbose' COMPILER='gcc' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh - Mesos ReviewBot On March 11, 2016, 7:50 a.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44694/ > ----------------------------------------------------------- > > (Updated March 11, 2016, 7:50 a.m.) > > > Review request for mesos, Joris Van Remoortere, Michael Park, and Till > Toenshoff. > > > Bugs: MESOS-4903 > https://issues.apache.org/jira/browse/MESOS-4903 > > > Repository: mesos > > > Description > ------- > > As long as the module manifest(s) being loaded don't conflict with the > already loaded manifests, the module manager will allow multiple calls > to `load()`. > > > Diffs > ----- > > src/module/manager.hpp 89dd06097a355c6455e8f8af7d3d98847a304c90 > src/module/manager.cpp 6ae99504005581b22a44768949b1d305cec517d9 > src/tests/module_tests.cpp 497ed87bddce23bb457373da40dc87111c04136d > > Diff: https://reviews.apache.org/r/44694/diff/ > > > Testing > ------- > > Added new tests and ran make check. > > > Thanks, > > Kapil Arya > >