----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66062/#review199658 -----------------------------------------------------------
Hi Feró, Thank you for improving our documentation! The content looks good to me, there is only one sytax error you need to fix and we can commmit it. Szabolcs src/docs/user/import.txt Lines 802 (patched) <https://reviews.apache.org/r/66062/#comment280030> This should be removed otherwise ant docs throws this error: asciidoc: ERROR: import.txt: line 810: [blockdef-listing] missing closing delimiter - Szabolcs Vasas On March 14, 2018, 11:15 a.m., Fero Szabo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66062/ > ----------------------------------------------------------- > > (Updated March 14, 2018, 11:15 a.m.) > > > Review request for Sqoop, Boglarka Egyed and Szabolcs Vasas. > > > Bugs: SQOOP-3292 > https://issues.apache.org/jira/browse/SQOOP-3292 > > > Repository: sqoop-trunk > > > Description > ------- > > Documentation for avro decimal padding i.e. the > sqoop.avro.logical_types.decimal.enable flag. > > I also added a few lines to mention sqoop.avro.logical_types.decimal.enable, > because the two goes hand in hand. > > Also fixed 2 typos. > > > Diffs > ----- > > src/docs/user/import.txt 330d5445 > > > Diff: https://reviews.apache.org/r/66062/diff/1/ > > > Testing > ------- > > > Thanks, > > Fero Szabo > >