----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48936/#review138507 -----------------------------------------------------------
Fix it, then Ship it! include/mesos/authorizer/acls.proto (lines 34 - 35) <https://reviews.apache.org/r/48936/#comment203697> The above note seems to be targetted towards the users/consumers of these messages whereas the new note is targeted towards Mesos developers, warning them to not break the pattern. I feel we should change the style here to make that more obvious. How about: ``` NOTE: When adding new actions to these proto definitions that entities in these messages are `required`. Absent ... ``` - Till Toenshoff On June 20, 2016, 12:12 a.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48936/ > ----------------------------------------------------------- > > (Updated June 20, 2016, 12:12 a.m.) > > > Review request for mesos, Alexander Rojas and Till Toenshoff. > > > Bugs: MESOS-5588 > https://issues.apache.org/jira/browse/MESOS-5588 > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > include/mesos/authorizer/acls.proto > f8e866b876c49e62a07f29edc66fc205ff33d618 > > Diff: https://reviews.apache.org/r/48936/diff/ > > > Testing > ------- > > None: Not a functional change. > > > Thanks, > > Alexander Rukletsov > >