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

Rahul Sharma commented on CRUNCH-16:
------------------------------------

I can think of one more way of solving this. This dependency is being  used in  
PTYPEs class. We can refactor the piece that uses this code to a separate class.
There will be one more implementation of this piece without the dependency. 
Both these will implementations will reside in separate src folders like 
src/main/CDH3/java and src/main/CHD4above/java. The src folders will be enabled 
on the profile that gets built. Hadoop1 will have this code and cdh3 version 
will not.

                
> Fix dependency versions
> -----------------------
>
>                 Key: CRUNCH-16
>                 URL: https://issues.apache.org/jira/browse/CRUNCH-16
>             Project: Crunch
>          Issue Type: Task
>            Reporter: Matthias Friedrich
>            Priority: Critical
>              Labels: maven
>
> Right now, the compile time classpath doesn't exactly match the runtime 
> classpath. For example, Hadoop CDH3u4 comes with slf4j-api-1.4.3 while Crunch 
> references slf4j-api-1.6.3. This could result in NoSuchMethodError and other 
> nasty problems.
> We have to make sure that version numbers match (check "mvn dependency:tree") 
> and that we don't ship artifacts that are already present on the runtime 
> classpath. If our dependencies introduce incompatible transitive 
> dependencies, we might have to downgrade them.
> Note: Maven's dependency resolution is order-dependent, see 
> http://maven.apache.org/guides/introduction/introduction-to-dependency-mechanism.html

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to