Re: Review Request 43711: Sqoop2: Enable SSL/TLS for API calls

2016-02-25 Thread Jarek Cecho
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43711/#review120743 --- Ship it! Ship It! - Jarek Cecho On Feb. 25, 2016, 12:04 a.m.

Re: Review Request 43711: Sqoop2: Enable SSL/TLS for API calls

2016-02-24 Thread Abraham Fine
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43711/ --- (Updated Feb. 25, 2016, 12:04 a.m.) Review request for Sqoop. Bugs: SQOOP-284

Re: Review Request 43711: Sqoop2: Enable SSL/TLS for API calls

2016-02-24 Thread Abraham Fine
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43711/ --- (Updated Feb. 24, 2016, 11:07 p.m.) Review request for Sqoop. Bugs: SQOOP-284

Re: Review Request 43711: Sqoop2: Enable SSL/TLS for API calls

2016-02-19 Thread Jarek Cecho
> On Feb. 19, 2016, 3:41 p.m., Jarek Cecho wrote: > > test/src/test/resources/connector-loading-tests-suite.xml, lines 21-31 > > > > > > Out of curiosity: Why this module change? > > Abraham Fine wrote: > the ssl

Re: Review Request 43711: Sqoop2: Enable SSL/TLS for API calls

2016-02-19 Thread Abraham Fine
> On Feb. 19, 2016, 3:41 p.m., Jarek Cecho wrote: > > test/src/test/resources/connector-loading-tests-suite.xml, lines 21-31 > > > > > > Out of curiosity: Why this module change? the ssltest needs to create its own i

Re: Review Request 43711: Sqoop2: Enable SSL/TLS for API calls

2016-02-19 Thread Jarek Cecho
--- 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.ja

Re: Review Request 43711: Sqoop2: Enable SSL/TLS for API calls

2016-02-18 Thread Abraham Fine
--- 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

Re: Review Request 43711: Sqoop2: Enable SSL/TLS for API calls

2016-02-18 Thread Abraham Fine
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43711/ --- (Updated Feb. 18, 2016, 6:36 p.m.) Review request for Sqoop. Bugs: SQOOP-2843

Re: Review Request 43711: Sqoop2: Enable SSL/TLS for API calls

2016-02-17 Thread Abraham Fine
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43711/ --- (Updated Feb. 18, 2016, 5:41 a.m.) Review request for Sqoop. Bugs: SQOOP-2843

Review Request 43711: Sqoop2: Enable SSL/TLS for API calls

2016-02-17 Thread Abraham Fine
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43711/ --- Review request for Sqoop. Bugs: SQOOP-2843 https://issues.apache.org/jira/b