-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39586/
-----------------------------------------------------------

(Updated Oct. 30, 2015, 7:29 a.m.)


Review request for Sqoop.


Bugs: SQOOP-2634
    https://issues.apache.org/jira/browse/SQOOP-2634


Repository: sqoop-sqoop2


Description
-------

Currently Sqoop 2 has already provided the ability to config jar dependencies 
with property "org.apache.sqoop.classpath.extra". The limitation of this 
property is that we have to put all the dependencies together. It can't express 
jar dependencies for a specified connector. This capacity is useful as some 
connectors may have conflict jar dependencies. Put all the dependencies from 
different connectors together may cause problems.


Diffs (updated)
-----

  common/src/main/java/org/apache/sqoop/error/code/ConnectorError.java 2f17d95 
  common/src/main/java/org/apache/sqoop/utils/ContextUtils.java 25d6fcb 
  
connector/connector-sdk/src/main/java/org/apache/sqoop/connector/spi/SqoopConnector.java
 85ba8be 
  core/src/main/java/org/apache/sqoop/connector/ConnectorHandler.java 1899bb7 
  core/src/main/java/org/apache/sqoop/connector/ConnectorManager.java f606950 
  core/src/main/java/org/apache/sqoop/connector/ConnectorManagerUtils.java 
9f9be57 
  core/src/main/java/org/apache/sqoop/connector/ConnectorUtils.java 
PRE-CREATION 
  core/src/main/java/org/apache/sqoop/core/ConfigurationConstants.java a33bc05 
  core/src/main/java/org/apache/sqoop/core/SqoopConfiguration.java 3c2f67e 
  core/src/main/java/org/apache/sqoop/driver/JobManager.java 15ca796 
  core/src/main/java/org/apache/sqoop/driver/JobRequest.java cfa45b2 
  core/src/test/java/org/apache/sqoop/connector/TestConnectorManagerUtils.java 
423b3df 
  core/src/test/java/org/apache/sqoop/connector/TestConnectorUtils.java 
PRE-CREATION 
  dist/src/main/conf/sqoop.properties 2895530 
  execution/mapreduce/src/main/java/org/apache/sqoop/job/MRJobConstants.java 
737ceda 
  
repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java
 ee5e8d1 
  
submission/mapreduce/src/main/java/org/apache/sqoop/submission/mapreduce/MapreduceSubmissionEngine.java
 f396783 

Diff: https://reviews.apache.org/r/39586/diff/


Testing
-------


Thanks,

Dian Fu

Reply via email to