Re: Review Request 54061: Augmented FrameworkInfo to support multiple roles.

2017-01-31 Thread Benjamin Mahler
> On Jan. 31, 2017, 6:32 p.m., David McLaughlin wrote: > > include/mesos/mesos.proto, line 256 > > > > > > Should this still be deprecated if master is advising not to use the > > roles field? We should have added

Re: Review Request 54061: Augmented FrameworkInfo to support multiple roles.

2017-01-31 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54061/#review163686 --- include/mesos/mesos.proto (line 256)

Re: Review Request 54061: Augmented FrameworkInfo to support multiple roles.

2016-11-30 Thread Benjamin Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54061/#review157485 --- Ship it! We should include some warnings that this isn't

Re: Review Request 54061: Augmented FrameworkInfo to support multiple roles.

2016-11-30 Thread Jay Guo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54061/ --- (Updated Nov. 30, 2016, 9:58 a.m.) Review request for mesos, Benjamin Mahler,