mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Vinod Kone <vinodk...@gmail.com>
Subject Re: Review Request 48272: Added 1.0 changed to upgrades.md.
Date Mon, 06 Jun 2016 15:54:34 GMT


> On June 6, 2016, 2:52 p.m., Vinod Kone wrote:
> > docs/upgrades.md, line 173
> > <https://reviews.apache.org/r/48272/diff/1/?file=1406841#file1406841line173>
> >
> >     This section seems like duplicate of what we have in CHANGELOG. What do you
think about removing this text from here and just point the table entries above to CHANGELOG?
> 
> Joerg Schad wrote:
>     I actually considered that, but decided against it for the following reason:
>     IMO this document is for operators who want to understand what might be problematic
when upgrading between different version (which by natures duplicates some information from
the CHANGELOG). Also the table should be just optional, so those changes should be mentioned
in the "0.28 ->  1.0" section as well as in the table.

Why can't operators look at CHANGELOG? Everything in the Deprecations section in CHANGELOG
should be something the operators should be aware of. Is there stuff in upgrades.md that is
not in CHANGELOG?


- Vinod


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


On June 6, 2016, 3:26 p.m., Joerg Schad wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48272/
> -----------------------------------------------------------
> 
> (Updated June 6, 2016, 3:26 p.m.)
> 
> 
> Review request for mesos and Vinod Kone.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Mesos 1.0 changed need to be reflected in upgrades.md.
> 
> 
> Diffs
> -----
> 
>   docs/upgrades.md be6db7d1c70e893bf74075db4658d92fb57cfa79 
> 
> Diff: https://reviews.apache.org/r/48272/diff/
> 
> 
> Testing
> -------
> 
> checked as gist: https://gist.github.com/joerg84/2426fca01ef09bb4af6454afa28faa56
> 
> 
> Thanks,
> 
> Joerg Schad
> 
>


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