> On Aug. 10, 2015, 1:29 p.m., Timothy Chen wrote:
> > include/mesos/mesos.proto, line 1208
> > <https://reviews.apache.org/r/37307/diff/1/?file=1036508#file1036508line1208>
> >
> >     Should the user have the ability then to specify either a name or a id?
> >     
> >     If name is required then I don't see why specify an id at the same time?

The user should have the ability and flexibility to specify:

name + version + other things (arch, os, etc., version can be SemVar or 
conventional patterns such as 'latest' or 'live' or 'production')
name (use the default version)
name + id

Even though the id may be universally unique, current Appc spec and our 
discovery implementation (not checked in yet) don't support queries by the id 
alone. Therefore 'name' being a required field is fine with me.

Plus I think knowing the name is much less of a burden and source of 
inflexibility than the id.

The motivation is for the image maintainer / cluster operator to be able to 
update the base image (for new tasks) across the cluster without requiring the 
users to update the `id` for each individual task. I should put this in the 
ticket.


- Jiang Yan


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


On Aug. 10, 2015, 11:31 a.m., Jiang Yan Xu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37307/
> -----------------------------------------------------------
> 
> (Updated Aug. 10, 2015, 11:31 a.m.)
> 
> 
> Review request for mesos, Ian Downes, Jie Yu, Timothy Chen, and Vinod Kone.
> 
> 
> Bugs: MESOS-3192
>     https://issues.apache.org/jira/browse/MESOS-3192
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> See the ticket for details.
> 
> 
> Diffs
> -----
> 
>   include/mesos/mesos.proto 80f56ac2178b24ff19f57c1ace13f567843c7807 
> 
> Diff: https://reviews.apache.org/r/37307/diff/
> 
> 
> Testing
> -------
> 
> N/A.
> 
> 
> Thanks,
> 
> Jiang Yan Xu
> 
>

Reply via email to