Re: Review Request 64726: Renamed operation protos for consistency.

2017-12-20 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64726/#review194271 --- FAIL: mesos-java failed to build. Reviews applied: `['64726']`

Re: Review Request 64726: Renamed operation protos for consistency.

2017-12-20 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64726/#review194268 --- Ship it! Ship It! - Greg Mann On Dec. 20, 2017, 7:21 p.m.,

Re: Review Request 64726: Renamed operation protos for consistency.

2017-12-20 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64726/#review194266 --- Ship it! Ship It! - Jie Yu On Dec. 20, 2017, 7:21 p.m.,

Re: Review Request 64726: Renamed operation protos for consistency.

2017-12-20 Thread Gaston Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64726/ --- (Updated Dec. 20, 2017, 11:21 a.m.) Review request for mesos, Chun-Hung Hsiao,

Re: Review Request 64726: Renamed operation protos for consistency.

2017-12-20 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64726/#review194261 --- include/mesos/mesos.proto Line 2279 (original), 2277 (patched)

Re: Review Request 64726: Renamed operation protos for consistency.

2017-12-19 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64726/#review194209 --- FAIL: mesos-java failed to build. Reviews applied: `['64726']`

Re: Review Request 64726: Renamed operation protos for consistency.

2017-12-19 Thread Gaston Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64726/ --- (Updated Dec. 19, 2017, 6:53 p.m.) Review request for mesos, Chun-Hung Hsiao,

Re: Review Request 64726: Renamed operation protos for consistency.

2017-12-19 Thread Gaston Kleiman
> On Dec. 19, 2017, 5:38 p.m., Greg Mann wrote: > > include/mesos/v1/scheduler/scheduler.proto > > Line 250 (original), 250 (patched) > > > > > > Can probably elaborate the comment here now that the name is

Re: Review Request 64726: Renamed operation protos for consistency.

2017-12-19 Thread Gaston Kleiman
> On Dec. 19, 2017, 5:38 p.m., Greg Mann wrote: > > include/mesos/mesos.proto > > Lines 723-741 (original), 723-741 (patched) > > > > > > I don't think we need to make any changes in `ExecutorInfo`. Here and > > in

Re: Review Request 64726: Renamed operation protos for consistency.

2017-12-19 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64726/#review194196 --- include/mesos/mesos.proto Lines 723-741 (original), 723-741

Re: Review Request 64726: Renamed operation protos for consistency.

2017-12-19 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64726/#review194197 --- FAIL: mesos-java failed to build. Reviews applied: `['64726']`

Re: Review Request 64726: Renamed operation protos for consistency.

2017-12-19 Thread Gaston Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64726/ --- (Updated Dec. 19, 2017, 4:57 p.m.) Review request for mesos, Chun-Hung Hsiao,

Re: Review Request 64726: Renamed operation protos for consistency.

2017-12-19 Thread Chun-Hung Hsiao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64726/#review194195 --- include/mesos/mesos.proto Line 1911 (original), 1911 (patched)

Re: Review Request 64726: Renamed operation protos for consistency.

2017-12-19 Thread Chun-Hung Hsiao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64726/#review194194 --- include/mesos/v1/resource_provider/resource_provider.proto Line