mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Joseph Wu" <>
Subject Re: Review Request 41004: Logger Module: Introduce the ContainerLogger module.
Date Sat, 12 Dec 2015 00:39:42 GMT

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

(Updated Dec. 11, 2015, 4:39 p.m.)

Review request for mesos, Benjamin Hindman, Cody Maloney, and Artem Harutyunyan.


Rename from ExecutorLogger to ContainerLogger.

Summary (updated)

Logger Module: Introduce the ContainerLogger module.

Bugs: MESOS-4087

Repository: mesos

Description (updated)

Modularizes the `ContainerLogger` interface and adds the `SandboxContainerLogger` as the default.

Diffs (updated)

  include/mesos/module/container_logger.hpp PRE-CREATION 
  src/ 9762f8567e32d70b8df2d694a1fef5c692fc730e 
  src/examples/test_container_logger_module.cpp PRE-CREATION 
  src/module/manager.cpp 1f04790510a2ab9ccd6907fd01be192f52ee90c6 
  src/slave/container_logger.cpp PRE-CREATION 
  src/tests/module.hpp e46ed12c80707bf44ceef3ed1a4eb2f321ce10f6 
  src/tests/module.cpp e272bf0eccb61ae54440ec79adac8efad804c828 


Testing (updated)


Tests are modified and run later in the review chain.


Joseph Wu

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