Re: Review Request 56742: Updated CHANGELOG for Mesos 1.2.0 release.

2017-02-23 Thread Adam B
> On Feb. 23, 2017, 4:39 p.m., Benjamin Bannier wrote: > > We simultanously use _unified containerizer_ and _Mesos containerizer_ here > > to describe 1.2.0 changes. I think picking a single name, at least in > > high-level descriptions, might be less confusing. > > Adam B wrote: > $ grep

Re: Review Request 56742: Updated CHANGELOG for Mesos 1.2.0 release.

2017-02-23 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56742/ --- (Updated Feb. 23, 2017, 5:01 p.m.) Review request for mesos, Jie Yu and Vinod K

Re: Review Request 56742: Updated CHANGELOG for Mesos 1.2.0 release.

2017-02-23 Thread Adam B
> On Feb. 23, 2017, 4:39 p.m., Benjamin Bannier wrote: > > We simultanously use _unified containerizer_ and _Mesos containerizer_ here > > to describe 1.2.0 changes. I think picking a single name, at least in > > high-level descriptions, might be less confusing. $ grep -i "unified containerize

Re: Review Request 56742: Updated CHANGELOG for Mesos 1.2.0 release.

2017-02-23 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56742/#review166621 --- We simultanously use _unified containerizer_ and _Mesos containeri

Re: Review Request 56742: Updated CHANGELOG for Mesos 1.2.0 release.

2017-02-23 Thread Adam B
> On Feb. 21, 2017, 11:11 a.m., Vinod Kone wrote: > > CHANGELOG, line 111 > > > > > > I think we can graduate this? cc @alexR > > Alexander Rukletsov wrote: > We haven't tested it on windows yet. Also we may be

Re: Review Request 56742: Updated CHANGELOG for Mesos 1.2.0 release.

2017-02-23 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56742/#review166617 --- CHANGELOG (line 131)

Re: Review Request 56742: Updated CHANGELOG for Mesos 1.2.0 release.

2017-02-23 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56742/#review166618 --- Ship it! Ship It! - Joerg Schad On Feb. 24, 2017, 12:04 a.m.

Re: Review Request 56742: Updated CHANGELOG for Mesos 1.2.0 release.

2017-02-23 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56742/ --- (Updated Feb. 23, 2017, 4:04 p.m.) Review request for mesos, Jie Yu and Vinod K

Re: Review Request 56742: Updated CHANGELOG for Mesos 1.2.0 release.

2017-02-23 Thread Alexander Rukletsov
> On Feb. 21, 2017, 7:11 p.m., Vinod Kone wrote: > > CHANGELOG, line 111 > > > > > > I think we can graduate this? cc @alexR We haven't tested it on windows yet. Also we may be changing the protos in the nearest fu

Re: Review Request 56742: Updated CHANGELOG for Mesos 1.2.0 release.

2017-02-21 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56742/#review166203 --- Fix it, then Ship it! CHANGELOG (line 108)

Re: Review Request 56742: Updated CHANGELOG for Mesos 1.2.0 release.

2017-02-20 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56742/#review166088 --- Patch looks great! Reviews applied: [56742] Passed command: expo

Re: Review Request 56742: Updated CHANGELOG for Mesos 1.2.0 release.

2017-02-20 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56742/ --- (Updated Feb. 20, 2017, 1:22 a.m.) Review request for mesos, Jie Yu and Vinod K

Re: Review Request 56742: Updated CHANGELOG for Mesos 1.2.0 release.

2017-02-16 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56742/#review165830 --- Bad review! Reviews applied: [] Error: No reviewers specified. P