[ 
https://issues.apache.org/jira/browse/SQOOP-2634?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15004530#comment-15004530
 ] 

Jarek Jarcec Cecho commented on SQOOP-2634:
-------------------------------------------

If we can configure the shade plugin to not relocate the code, then I'm fine 
with using it.

{quote}
If the dependency conflict with existing libraries in lib/, loading it with 
method Initializer.getJars will cause problems. Thoughts?
{quote}

I feel that this is not a new problem here - connector can already require one 
dependency twice which will lead to undefined state. Hence I'm fine with 
keeping this behavior and simply expecting that connector developers knows what 
they are doing. I guess that we can put some smarter code detecting such 
scenarios and at least warn about them if needed.

> Sqoop2: Allow connectors to express jar dependencies
> ----------------------------------------------------
>
>                 Key: SQOOP-2634
>                 URL: https://issues.apache.org/jira/browse/SQOOP-2634
>             Project: Sqoop
>          Issue Type: Sub-task
>            Reporter: Dian Fu
>            Assignee: Dian Fu
>             Fix For: 1.99.7
>
>         Attachments: SQOOP-2634.001.patch, SQOOP-2634.002.patch, 
> SQOOP-2634.003.patch, SQOOP-2634.004.patch, SQOOP-2634.005.patch, 
> SQOOP-2634.006.patch, SQOOP-2634.007.patch, SQOOP-2634.008.patch, 
> SQOOP-2634.009.patch, SQOOP-2634.010.patch, design-doc-v1.pdf, 
> design-doc-v2.pdf, design-doc-v3.pdf
>
>
> 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.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to