mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Neil Conway <neil.con...@gmail.com>
Subject Re: Review Request 46038: Added recommended production config to the docs.
Date Mon, 11 Apr 2016 18:23:09 GMT

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



Providing recommended values without justifying why they are "recommended" seems a bit problematic
to me, because it encourages people to blindly copy the recommended configuration without
understanding whether it is applicable for their environment. What do you think about adding
an "explanation" column with a short justification for why a particular value is recommended?

- Neil Conway


On April 11, 2016, 5:16 p.m., Greg Mann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46038/
> -----------------------------------------------------------
> 
> (Updated April 11, 2016, 5:16 p.m.)
> 
> 
> Review request for mesos, Jie Yu and Neil Conway.
> 
> 
> Bugs: MESOS-5064
>     https://issues.apache.org/jira/browse/MESOS-5064
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Added recommended production config to the docs.
> 
> 
> Diffs
> -----
> 
>   docs/operational-guide.md 5ae7ede3f500380a78364d5c3da2c4cea75e04c5 
> 
> Diff: https://reviews.apache.org/r/46038/diff/
> 
> 
> Testing
> -------
> 
> Viewed with the Mesos website container: https://github.com/mesosphere/mesos-website-container
> 
> 
> Thanks,
> 
> Greg Mann
> 
>


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