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

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
-----

  common/src/main/java/org/apache/sqoop/utils/ContextUtils.java 25d6fcb 
  core/src/main/java/org/apache/sqoop/connector/ConnectorManager.java 285e893 
  core/src/main/java/org/apache/sqoop/connector/ConnectorManagerUtils.java 
522ed08 
  core/src/main/java/org/apache/sqoop/connector/ConnectorUtils.java 
PRE-CREATION 
  core/src/main/java/org/apache/sqoop/core/ConfigurationConstants.java f49fca3 
  core/src/main/java/org/apache/sqoop/core/SqoopConfiguration.java 3c2f67e 
  core/src/test/java/org/apache/sqoop/connector/TestConnectorManagerUtils.java 
fd2d1b4 
  core/src/test/java/org/apache/sqoop/connector/TestConnectorUtils.java 
PRE-CREATION 
  dist/src/main/conf/sqoop.properties 7c1ec18 
  execution/mapreduce/src/main/java/org/apache/sqoop/job/MRJobConstants.java 
b7aa8c6 
  
repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java
 0302150 
  
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