----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27577/#review59778 -----------------------------------------------------------
Ship it! Looks great, just few comments. docs/src/site/sphinx/Upgrade.rst <https://reviews.apache.org/r/27577/#comment101080> Not sure we should mention "configurables" here - its an "internals" term that won't be clear to users. docs/src/site/sphinx/Upgrade.rst <https://reviews.apache.org/r/27577/#comment101081> Not just schema upgrade, right? the upgrade may modify content as well. BTW. Since we are re-writing the docs - maybe advise on taking backups prior to upgrade? - Gwen Shapira On Nov. 4, 2014, 5:21 p.m., Veena Basavaraj wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27577/ > ----------------------------------------------------------- > > (Updated Nov. 4, 2014, 5:21 p.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-1673 > https://issues.apache.org/jira/browse/SQOOP-1673 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > see jira > > https://issues.apache.org/jira/browse/SQOOP-1673 > > > Diffs > ----- > > docs/src/site/sphinx/Upgrade.rst aa3316c > > Diff: https://reviews.apache.org/r/27577/diff/ > > > Testing > ------- > > > Thanks, > > Veena Basavaraj > >
