mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Park" <mcyp...@gmail.com>
Subject Re: Review Request 32139: Add 'Resource::ReservationInfo' protobuf message.
Date Wed, 25 Mar 2015 18:48:45 GMT


> On March 24, 2015, 7:25 a.m., Timothy Chen wrote:
> > include/mesos/mesos.proto, line 397
> > <https://reviews.apache.org/r/32139/diff/2/?file=899198#file899198line397>
> >
> >     Not sure this is the right place to comment on the valid specifications of both
role and reservation. There is a user guide added right? We should refer to that instead.
> 
> Michael Park wrote:
>     Hm, the user guide doesn't exist in the repo. Maybe we just say something like: "Refer
to the reservations user guide for the valid states and transitions of (role, reservation)
pair." and add in the links together with the user guide later?

It seems weird to refer to a user guide that doesn't exist yet. I'll add the comment here
with the link to the user guide when the user guide is ready to be added to the repo.


- Michael


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


On March 25, 2015, 6:47 p.m., Michael Park wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32139/
> -----------------------------------------------------------
> 
> (Updated March 25, 2015, 6:47 p.m.)
> 
> 
> Review request for mesos, Alexander Rukletsov, Ben Mahler, and Jie Yu.
> 
> 
> Bugs: MESOS-2475
>     https://issues.apache.org/jira/browse/MESOS-2475
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> The beginning of `Resource::ReservationInfo`. This patch only introduces the `framework_id`
field. We don't need the required `principal` field yet and will be introduced together with
the "unreserve" ACLs. An optional `id` may be introduced in the future as well.
> 
> 
> Diffs
> -----
> 
>   include/mesos/mesos.proto 3c592d5ab3092ecbeddfaff95e0c1addc3ac58f8 
> 
> Diff: https://reviews.apache.org/r/32139/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Michael Park
> 
>


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