----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43711/#review119889 -----------------------------------------------------------
core/src/main/java/org/apache/sqoop/security/SecurityConstants.java (lines 192 - 201) <https://reviews.apache.org/r/43711/#comment181282> Nit: It seems that we either use dot to separate names or underscores rather then just removing spaces. Perhaps we should stay consistent? test/src/test/resources/connector-loading-tests-suite.xml (lines 21 - 31) <https://reviews.apache.org/r/43711/#comment181283> Out of curiosity: Why this module change? - Jarek Cecho On Feb. 18, 2016, 6:45 p.m., Abraham Fine wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43711/ > ----------------------------------------------------------- > > (Updated Feb. 18, 2016, 6:45 p.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-2843 > https://issues.apache.org/jira/browse/SQOOP-2843 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > Sqoop2: Enable SSL/TLS for API calls > > > Diffs > ----- > > core/src/main/java/org/apache/sqoop/security/SecurityConstants.java > 6f32e04be6fc47051e7c0cc90a881cb872cb3934 > dist/src/main/conf/sqoop.properties > 620146d5c8a2f736d5d139e6452b0165cbdfaa17 > server/src/main/java/org/apache/sqoop/server/SqoopJettyServer.java > 2c4cb7a13aacd1e4e3222d54fb3b1ef518aa2ebb > server/src/main/java/org/apache/sqoop/server/common/ServerError.java > 1b021cf27defee1493ad620466a89b9fae1b5bb5 > test/src/main/java/org/apache/sqoop/test/kdc/MiniKdcRunner.java > 83f960b87987d1c1ed8f0161616e631f63f14543 > test/src/main/java/org/apache/sqoop/test/utils/SecurityUtils.java > PRE-CREATION > test/src/main/java/org/apache/sqoop/test/utils/SqoopUtils.java > 6614b19932c304aa40500fcaedd974b1b136caec > > test/src/test/java/org/apache/sqoop/integration/connectorloading/BlacklistedConnectorTest.java > 7f0575b49c58fcec1c69069f25929411a6c7ba12 > > test/src/test/java/org/apache/sqoop/integration/connectorloading/ClasspathTest.java > 87f0eb1966de4d7ea9b1c37b881cad0ae7a88fa4 > > test/src/test/java/org/apache/sqoop/integration/connectorloading/ConnectorClasspathIsolationTest.java > a82a4daea607bee4e78bf2a649b7c9d5228448be > > test/src/test/java/org/apache/sqoop/integration/serverproperties/SslTest.java > PRE-CREATION > test/src/test/resources/connector-loading-tests-suite.xml > 02c1df3aa7bc6f4fa2ddf94ce436bce5d07ceb72 > > Diff: https://reviews.apache.org/r/43711/diff/ > > > Testing > ------- > > passing integration tests > > > Thanks, > > Abraham Fine > >
