mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Mahler <bmah...@apache.org>
Subject Re: Review Request 57106: Added `FrameworkInfo` to `UpdateFrameworkMessage`.
Date Tue, 28 Feb 2017 02:42:13 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57106/#review166992
-----------------------------------------------------------




src/messages/messages.proto (lines 574 - 575)
<https://reviews.apache.org/r/57106/#comment239096>

    Just one thing, looking at the subsequent patch, it appears that the agent updates the
entire framework info, not just capabilities. So let's remove this TODO? Is there anything
you want to call out here that won't be updated correctly by the agent if it were to change?


- Benjamin Mahler


On Feb. 27, 2017, 10:11 p.m., Michael Park wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/57106/
> -----------------------------------------------------------
> 
> (Updated Feb. 27, 2017, 10:11 p.m.)
> 
> 
> Review request for mesos and Benjamin Mahler.
> 
> 
> Bugs: MESOS-6627
>     https://issues.apache.org/jira/browse/MESOS-6627
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Taken verbatim from the [design doc](https://docs.google.com/document/d/1n0ZtRL49m3qpAeB4beEEhjTXQS-nYAbgRG9g44fn4WY).
> 
> 
> Diffs
> -----
> 
>   src/messages/messages.proto 2da89a8e6893b6be1faa89a6e1011f80fe643698 
> 
> Diff: https://reviews.apache.org/r/57106/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Michael Park
> 
>


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