mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bartek Plotka" <>
Subject Re: Review Request 34748: Defined protobuf for usage returned by Resource Monitor. Reused ResourceUsage. Pass ResourceUsage to ResourceEstimator
Date Tue, 02 Jun 2015 19:40:33 GMT

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

(Updated June 2, 2015, 7:40 p.m.)

Review request for mesos, Ben Mahler, Jie Yu, Joris Van Remoortere, Niklas Nielsen, Szymon
Konefal, and Vinod Kone.


First part of issues addressed.

Bugs: MESOS-2772

Repository: mesos


Changed ResourceMonitor to use ResourceUsage instead of ResourceMonitor::Usage.
Reused old ResoureUsage in message mesos.proto

NOTE: That is required for modules which need to fetch ResourceUsage e.g ResourceEstimator
and QoSController.
Discussed that message ResourceUsage in mesos.proto is free to reuse and modify, since it's
not used anywhere else.

Diffs (updated)

  include/mesos/mesos.proto a66888916dc90e476c8bb20e67e3f6b08c47fb99 



make check


Bartek Plotka

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