----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7651/#review12912 -----------------------------------------------------------
I have two minor comments as below. Please let me know what you think. src/docs/user/troubleshooting.txt <https://reviews.apache.org/r/7651/#comment27762> Can you change this to "as it is due to a Java bug ..."? Note that I dropped the "as" before the "due". src/docs/user/troubleshooting.txt <https://reviews.apache.org/r/7651/#comment27763> Can you change this line as follows? ", or +/dev/urandom+ will be ignored and substituted by +/dev/random+." - Cheolsoo Park On Oct. 30, 2012, 3:53 p.m., Jarek Cecho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7651/ > ----------------------------------------------------------- > > (Updated Oct. 30, 2012, 3:53 p.m.) > > > Review request for Sqoop. > > > Description > ------- > > I've linked relevant Java bug and explained why it's needed. > > > This addresses bug SQOOP-635. > https://issues.apache.org/jira/browse/SQOOP-635 > > > Diffs > ----- > > src/docs/user/troubleshooting.txt d46a886c98be0f04303218c922f080ecab192707 > > Diff: https://reviews.apache.org/r/7651/diff/ > > > Testing > ------- > > I've build docs and verified that they look ok. > > > Thanks, > > Jarek Cecho > >
