----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46501/#review132234 -----------------------------------------------------------
docs/authorization.md (line 12) <https://reviews.apache.org/r/46501/#comment196413> I'd avoid passive voice here: "The next section explores the concepts necessary to successfully configure the local authorizer". Personally I'd opt for removing the rest of the paragraph. Typos: "sections", "finalized", "discussion". Personally I would opt for using italics around "local authorizer" only the first time the term is used, and then not using italics for subsequent uses. docs/authorization.md (line 14) <https://reviews.apache.org/r/46501/#comment196416> "The final section briefly discusses how to implement a custom authorizer." docs/authorization.md (line 462) <https://reviews.apache.org/r/46501/#comment196414> I'd move this section up before the examples. - Neil Conway On May 9, 2016, 8:49 a.m., Alexander Rojas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46501/ > ----------------------------------------------------------- > > (Updated May 9, 2016, 8:49 a.m.) > > > Review request for mesos, Adam B, Alexander Rukletsov, Greg Mann, and Neil > Conway. > > > Bugs: MESOS-4785 > https://issues.apache.org/jira/browse/MESOS-4785 > > > Repository: mesos > > > Description > ------- > > The API of the authorization has been changing constantly over the > last few versions. This patch attempts to update the documentation to > the those changes into account. > > > Diffs > ----- > > docs/authorization.md 0db5c345b3239814b3b9d2e8a87601ff69d0f869 > > Diff: https://reviews.apache.org/r/46501/diff/ > > > Testing > ------- > > > Thanks, > > Alexander Rojas > >
