> On Oct. 16, 2014, 11:03 p.m., Joshua Cohen wrote:
> > docs/client-cluster-configuration.md, line 34
> > <https://reviews.apache.org/r/26834/diff/1/?file=723472#file723472line34>
> >
> >     I think an argument could be made for removing this configuration 
> > entirely and just hardcoding this into the client. I don't think there's a 
> > use case where we'd want these commands to talk to any run other than the 
> > latest run (correct me if I'm wrong), however having this here does protect 
> > us against having to redeploy the client in the event that mesos changes 
> > its path structure.
> 
> Kevin Sweeney wrote:
>     I'd be +1 to dropping this parameter entirely, either in this patch or a 
> followup

At a bare minimum make latest the default and make this optional.


- Jay


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


On Oct. 16, 2014, 10:56 p.m., Joshua Cohen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26834/
> -----------------------------------------------------------
> 
> (Updated Oct. 16, 2014, 10:56 p.m.)
> 
> 
> Review request for Aurora, Maxim Khutornenko and Bill Farner.
> 
> 
> Bugs: AURORA-846
>     https://issues.apache.org/jira/browse/AURORA-846
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Add client cluster configuration docs.
> 
> 
> Diffs
> -----
> 
>   docs/client-cluster-configuration.md PRE-CREATION 
>   docs/client-commands.md f61fc661d77345950d71bd3606dbe3d1488e9e5a 
> 
> Diff: https://reviews.apache.org/r/26834/diff/
> 
> 
> Testing
> -------
> 
> Rendered here: 
> https://github.com/jcohen/incubator-aurora/blob/jcohen/docs/clusters.json/docs/client-cluster-configuration.md
> 
> 
> Thanks,
> 
> Joshua Cohen
> 
>

Reply via email to