mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Kapil Arya <>
Subject Re: Review Request 44694: Enabled multiple calls to ModuleManager::load().
Date Fri, 11 Mar 2016 02:51:33 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated March 10, 2016, 9:51 p.m.)

Review request for mesos, Joris Van Remoortere, Michael Park, and Till Toenshoff.


Removed Parameter comparison

Bugs: MESOS-4903

Repository: mesos

Description (updated)

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 (updated)

  src/module/manager.hpp 89dd06097a355c6455e8f8af7d3d98847a304c90 
  src/module/manager.cpp 6ae99504005581b22a44768949b1d305cec517d9 
  src/tests/module_tests.cpp 497ed87bddce23bb457373da40dc87111c04136d 



Added new tests and ran make check.


Kapil Arya

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