mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Marco Massenzio" <>
Subject Re: Review Request 33376: MESOS-2633 Moved struct Framework methods to their own implementation class.
Date Wed, 22 Apr 2015 20:35:13 GMT

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

(Updated April 22, 2015, 8:35 p.m.)

Review request for mesos and Joris Van Remoortere.


reverted changes

Bugs: MESOS-2633

Repository: mesos


Created new file framework.cpp containing all the methods' implementations for the `Framework`
class (declared in master/master.hpp)

Declared `operator ==` for Task in type_utils.hpp 
(it was implemented before, but not declared in the header file);

Refactored all the LOG(WARNING) to a single utility method.

Diffs (updated)

  include/mesos/type_utils.hpp cdf5864389a72002b538c263d70bcade2bdffa45 
  src/ 93c7c8a807a33ab639be6289535bbd32022aa85b 
  src/master/framework.cpp PRE-CREATION 
  src/master/master.hpp 550d2c50cd01aa5830a7e7e03ab4a0240c221b15 



All tests (make check) pass.


Marco Massenzio

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