[jira] [Commented] (PIG-3540) e2e test for tez

["Cheolsoo Park (JIRA)" <ji...@apache.org>]

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

Cheolsoo Park commented on PIG-3540:
------------------------------------

[~daijy], thank you for the patch! I have two comments:
# Can you add a top-level ant target? Perhaps "test-e2e-tez"? (I am already using "test-tez" for unit tests in PIG-3535.)
# Can you delete the following lines?
{code:title=test/e2e/pig/drivers/TestDriverPig.pm}
+print $log "hehehehehe\n";
+print $log $ENV{'PIG_CLASSPATH'};
+print $log "hahahahaha\n";
{code}
I think the followings are not applicable.
{code:title=test/e2e/pig/build.xml}
+    <property name="hadoop.mapred.local.dir" value="/tmp/hadoop/mapred/local"/>
...
+      <env key="HADOOP_MAPRED_LOCAL_DIR" value="${hadoop.mapred.local.dir}"/>
{code}

Otherwise looks good to me. Thanks!

> e2e test for tez
> ----------------
>
>                 Key: PIG-3540
>                 URL: https://issues.apache.org/jira/browse/PIG-3540
>             Project: Pig
>          Issue Type: Sub-task
>          Components: tez
>            Reporter: Daniel Dai
>            Assignee: Daniel Dai
>             Fix For: tez-branch
>
>         Attachments: PIG-3540-1.patch
>
>
> Need to add e2e tests for tez. Should be able to compare the tez run with mapreduce run.



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