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

Rohini Palaniswamy commented on OOZIE-1412:
-------------------------------------------

bq.  -1 the patch does not add/modify any testcase
  Its a pom.xml change

bq. -1 the patch seems to introduce 13 new javac warning(s)
  These are maven warnings and not java warnings. * exclusion is a unsupported 
feature but works. Considering the number of exclusions, I thought best to go 
with it. But on after thought should have gone with copy dependencies plugin to 
be more clean. Will give that a shot and put up a new patch.

bq. https://builds.apache.org/job/oozie-trunk-precommit-build/587/testReport/
  Test failures not related.


                
> Webapp contains all sharedlib dependencies after launcher refactor
> ------------------------------------------------------------------
>
>                 Key: OOZIE-1412
>                 URL: https://issues.apache.org/jira/browse/OOZIE-1412
>             Project: Oozie
>          Issue Type: Bug
>            Reporter: Rohini Palaniswamy
>            Assignee: Rohini Palaniswamy
>             Fix For: trunk
>
>         Attachments: OOZIE-1412-1.patch, OOZIE-1412-1.patch
>
>
>   With the launcher refactor, the webapp now includes sharelib as dependency. 
> Exclusions need to be there to prevent pig, hive and sqoop jars being 
> packaged as part of webapp

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to