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

Steve Loughran commented on HADOOP-10100:
-----------------------------------------

This patch also changed the junit dependencies. Was that intentional? If not it 
slipped through review & also needs to be in the title of the patch/changes

Pushing through potentially significant changes in dependencies and export in 
3h,  without even a chance of those people who are working on actually updating 
al 
# Please don't change multiple dependencies in one go as we are trying to do 
them one by one -with careful review. 
# This wasn't a critical "fix in hours our disaster" patch, so I'd have liked 
to have been given an opportunity to review -which I would have done by getting 
the junit part of the diff moved to a separate patch that also updated the 
junit version to 4.11

Now, was the junit patch an accident? If so could we roll it back and have a 
separate patch with the junit update, and a separate note in CHANGES.TXT?


> MiniKDC shouldn't use apacheds-all artifact
> -------------------------------------------
>
>                 Key: HADOOP-10100
>                 URL: https://issues.apache.org/jira/browse/HADOOP-10100
>             Project: Hadoop Common
>          Issue Type: Bug
>    Affects Versions: 2.3.0
>            Reporter: Robert Kanter
>            Assignee: Robert Kanter
>             Fix For: 2.3.0
>
>         Attachments: HADOOP-10100.patch, HADOOP-10100.patch
>
>
> The MiniKDC currently depends on the {{apacheds-all}} artifact:
> {code:xml}
>     <dependency>
>       <groupId>org.apache.directory.server</groupId>
>       <artifactId>apacheds-all</artifactId>
>       <version>2.0.0-M15</version>
>       <scope>compile</scope>
>     </dependency>
> {code}
> However, this artifact includes, inside of itself, a lot of other packages, 
> including antlr, ehcache, apache commons, and mina (you can see a full list 
> of the packages in the jar 
> [here|http://mvnrepository.com/artifact/org.apache.directory.server/apacheds-all/2.0.0-M15]).
>   This can be problematic if other projects (e.g. Oozie) try to use MiniKDC 
> and have a different version of one of those dependencies (in my case, 
> ehcache).  Because the packages are included inside the {{apacheds-all}} jar, 
> we can't override their version.  
> Instead, we should remove {{apacheds-all}} and use dependencies that only 
> include org.apache.directory.* packages; the other necessary dependencies 
> should be included normally.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to