----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40393/#review106956 -----------------------------------------------------------
Ship it! Ship It! - Abraham Fine On Nov. 17, 2015, 5:54 p.m., Jarek Cecho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40393/ > ----------------------------------------------------------- > > (Updated Nov. 17, 2015, 5:54 p.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-2694 > https://issues.apache.org/jira/browse/SQOOP-2694 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > Even though that the patch looks large, it's actually quite simple. All it > does is: > > * Moving files around to various directories so that it's easier to build the > tree structure for sphinx. > * Changes theme to one that have side panel > * Fixes links for new theme that actually have the side panel (to verify the > tree structure) > * Enables build failures if the docs can't be build properly > > I could probably separate the last two items to standalone JIRA, but I feel > that the changes are so related, that it's better to do that as part of one > commit. > > > Diffs > ----- > > docs/pom.xml 079e896 > docs/src/site/sphinx/BuildingSqoop2.rst 7fbbb6b > docs/src/site/sphinx/ClientAPI.rst 9626878 > docs/src/site/sphinx/CommandLineClient.rst 8c4c592 > docs/src/site/sphinx/Connector-FTP.rst cc10d68 > docs/src/site/sphinx/Connector-GenericJDBC.rst 347547d > docs/src/site/sphinx/Connector-HDFS.rst c44b1b6 > docs/src/site/sphinx/Connector-Kafka.rst b6bca14 > docs/src/site/sphinx/Connector-Kite.rst 414ad8a > docs/src/site/sphinx/Connector-SFTP.rst d25ea3f > docs/src/site/sphinx/ConnectorDevelopment.rst 0e8ea92 > docs/src/site/sphinx/DevEnv.rst 3b72e06 > docs/src/site/sphinx/Installation.rst 9d56875 > docs/src/site/sphinx/RESTAPI.rst 39aabc0 > docs/src/site/sphinx/Repository.rst 55daf2e > docs/src/site/sphinx/SecurityGuideOnSqoop2.rst 7194d3b > docs/src/site/sphinx/Sqoop5MinutesDemo.rst 19115a2 > docs/src/site/sphinx/Tools.rst fb0187a > docs/src/site/sphinx/Upgrade.rst 385c5ae > docs/src/site/sphinx/admin.rst PRE-CREATION > docs/src/site/sphinx/admin/Installation.rst PRE-CREATION > docs/src/site/sphinx/admin/Tools.rst PRE-CREATION > docs/src/site/sphinx/admin/Upgrade.rst PRE-CREATION > docs/src/site/sphinx/conf.py 6a9bf31 > docs/src/site/sphinx/dev.rst PRE-CREATION > docs/src/site/sphinx/dev/BuildingSqoop2.rst PRE-CREATION > docs/src/site/sphinx/dev/ClientAPI.rst PRE-CREATION > docs/src/site/sphinx/dev/ConnectorDevelopment.rst PRE-CREATION > docs/src/site/sphinx/dev/DevEnv.rst PRE-CREATION > docs/src/site/sphinx/dev/RESTAPI.rst PRE-CREATION > docs/src/site/sphinx/dev/Repository.rst PRE-CREATION > docs/src/site/sphinx/index.rst a18fad2 > docs/src/site/sphinx/security.rst PRE-CREATION > docs/src/site/sphinx/security/SecurityGuideOnSqoop2.rst PRE-CREATION > docs/src/site/sphinx/user.rst PRE-CREATION > docs/src/site/sphinx/user/CommandLineClient.rst PRE-CREATION > docs/src/site/sphinx/user/Connectors.rst PRE-CREATION > docs/src/site/sphinx/user/Sqoop5MinutesDemo.rst PRE-CREATION > docs/src/site/sphinx/user/connectors/Connector-FTP.rst PRE-CREATION > docs/src/site/sphinx/user/connectors/Connector-GenericJDBC.rst PRE-CREATION > docs/src/site/sphinx/user/connectors/Connector-HDFS.rst PRE-CREATION > docs/src/site/sphinx/user/connectors/Connector-Kafka.rst PRE-CREATION > docs/src/site/sphinx/user/connectors/Connector-Kite.rst PRE-CREATION > docs/src/site/sphinx/user/connectors/Connector-SFTP.rst PRE-CREATION > > Diff: https://reviews.apache.org/r/40393/diff/ > > > Testing > ------- > > > Thanks, > > Jarek Cecho > >
