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

Alan Gates commented on PIG-219:
--------------------------------

The general approach looks good.  One comment on the name PigTestCase.  This 
class is specific to tests that need to execute pig scripts.  Other tests 
should instead just extend TestCase.  It think that's fine, we should just 
rename PigTestCase to make it clear that it is only needed for tests that 
execute pig scripts.  Maybe something like PigExecTestCase.

Other than that, I think it looks fine to go ahead and convert the other tests 
that execute pig scripts to this so they run in local mode as well.



> Pig tests must cover local and mapreduce execution types
> --------------------------------------------------------
>
>                 Key: PIG-219
>                 URL: https://issues.apache.org/jira/browse/PIG-219
>             Project: Pig
>          Issue Type: Bug
>            Reporter: Mathieu Poumeyrol
>         Attachments: Test.v1.patch
>
>
> Followup of Local and MapReduce Test modes in pig-dev.

-- 
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