> On Oct. 16, 2014, 4: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.
I'd be +1 to dropping this parameter entirely, either in this patch or a followup - Kevin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26834/#review57030 ----------------------------------------------------------- On Oct. 16, 2014, 3: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, 3: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 > >
