[ 
https://issues.apache.org/jira/browse/PIG-1452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12894077#action_12894077
 ] 

Konstantin Boudnik commented on PIG-1452:
-----------------------------------------

Normally, it isn't a good idea at all to include any jars into a production jar 
file. If you need extra jars to be delivered they are better be delivered 
separately, i.e. through Ivy dependencies. So, I believe the build needs to be 
changed a bit especially in this line
{noformat}
+            <zipgroupfileset dir="${ivy.lib.dir}" includes="*.jar" />
{noformat}
While I am not suggesting how Pig needs to deliver its stuff having dependency 
jars packed into the same gigantic jar file doesn't seem right. 

> to remove hadoop20.jar from lib and use hadoop from the apache maven repo.
> --------------------------------------------------------------------------
>
>                 Key: PIG-1452
>                 URL: https://issues.apache.org/jira/browse/PIG-1452
>             Project: Pig
>          Issue Type: Improvement
>          Components: build
>    Affects Versions: 0.8.0
>            Reporter: Giridharan Kesavan
>            Assignee: Giridharan Kesavan
>             Fix For: 0.8.0
>
>         Attachments: PIG-1452.PATCH, PIG-1452V2.PATCH
>
>
> pig use ivy for dependency management. But still it uses hadoop20.jar from 
> the lib folder. 
> Now that we have the hadoop-0.20.2 artifacts available in the maven repo, pig 
> should leverage ivy for resolving/retrieving hadoop artifacts.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to