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




include/mesos/resource_provider/resource_provider.proto
Line 37 (original), 37 (patched)
<https://reviews.apache.org/r/64143/#comment270497>

    please update v1 proto too.



include/mesos/resource_provider/resource_provider.proto
Lines 38 (patched)
<https://reviews.apache.org/r/64143/#comment270496>

    See my comment in the following patch. Let's make the code consistent on 
naming.


- Jie Yu


On Nov. 29, 2017, 5:50 p.m., Greg Mann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64143/
> -----------------------------------------------------------
> 
> (Updated Nov. 29, 2017, 5:50 p.m.)
> 
> 
> Review request for mesos, Chun-Hung Hsiao, Gaston Kleiman, and Jie Yu.
> 
> 
> Bugs: MESOS-8186
>     https://issues.apache.org/jira/browse/MESOS-8186
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> The new event is used to send offer operation status udpate
> acknowledgements to resource providers.
> 
> 
> Diffs
> -----
> 
>   include/mesos/resource_provider/resource_provider.proto 
> 615a2ab8a00f4da3906ad0ae322a885c84b3574b 
>   src/resource_provider/storage/provider.cpp 
> 49c042cdb1837860aaedde2e48f318ed5ac8b1d1 
> 
> 
> Diff: https://reviews.apache.org/r/64143/diff/1/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Greg Mann
> 
>

Reply via email to