----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28834/#review64858 -----------------------------------------------------------
Ship it! This looks fine from a content perspective. I noticed that there are bullet point characters. Can we replace them with ReStructed Text bullet points? docs/src/site/sphinx/KerberosOnSqoop2.rst <https://reviews.apache.org/r/28834/#comment107658> Bullet points can be replaced with "-". For example: - All components which... The above will be replaced with an actual bullet point. - Abraham Elmahrek On Dec. 10, 2014, 7:10 a.m., richard zhou wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28834/ > ----------------------------------------------------------- > > (Updated Dec. 10, 2014, 7:10 a.m.) > > > Review request for Sqoop. > > > Repository: sqoop-sqoop2 > > > Description > ------- > > Given Kerberos has been implemented and sqoop2 now provides SPNEGO, it would > be nice to have a security guide which explains the following: > Features. > High level design. > Usage. > > > Diffs > ----- > > docs/src/site/sphinx/KerberosOnSqoop2.rst PRE-CREATION > > Diff: https://reviews.apache.org/r/28834/diff/ > > > Testing > ------- > > > Thanks, > > richard zhou > >
