You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pig.apache.org by "Daniel Dai (JIRA)" <ji...@apache.org> on 2009/02/01 22:09:59 UTC

[jira] Updated: (PIG-654) Optimize build.xml

     [ https://issues.apache.org/jira/browse/PIG-654?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Daniel Dai updated PIG-654:
---------------------------

    Attachment: PIG-654.patch

> Optimize build.xml
> ------------------
>
>                 Key: PIG-654
>                 URL: https://issues.apache.org/jira/browse/PIG-654
>             Project: Pig
>          Issue Type: Improvement
>          Components: impl
>    Affects Versions: types_branch
>            Reporter: Daniel Dai
>            Assignee: Daniel Dai
>            Priority: Trivial
>             Fix For: types_branch
>
>         Attachments: PIG-654.patch
>
>
> Build process can be faster. Here is the issues slow down the build process:
> 1. Should change test/org/apache/pig/test/utils/dotGraph/Dot.jjt to DOTParser.jjt. [jjtree] assumes the output file name the same with the input file name, otherwise, it will recompile every time
> 2. Delete src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POPostCombinerPackage.java and src/org/apache/pig/impl/plan/SplitIntroducer.java, both classes are empty and generates no class file. Ant will recompile them every time since it do not see the output class file.
> I do not know how to create a patch with file deletion and rename, so I describe the actions below:
> 1. mv test/org/apache/pig/test/utils/dotGraph/Dot.jjt test/org/apache/pig/test/utils/dotGraph/DOTParser.jjt
> 2. rm src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POPostCombinerPackage.java
> 3. rm src/org/apache/pig/impl/plan/SplitIntroducer.java
> 4. apply patch to change build.xml (Dot.jjt -> DOTParser.jjt)

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