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

Andrey Klochkov commented on PIG-2682:
--------------------------------------

How about adding a parameter "expected_outfiles_count" into the hash for tests 
which do that? If the parameter is presented then TestDriverPig.countStores 
would just pick this value instead of trying to parse the script. It's not 
going to be used often so an additional inconvenience should be acceptable.
                
> pig harness does not correctly count the number of stores for multiple 
> invocations of the same macro
> ----------------------------------------------------------------------------------------------------
>
>                 Key: PIG-2682
>                 URL: https://issues.apache.org/jira/browse/PIG-2682
>             Project: Pig
>          Issue Type: Test
>          Components: e2e harness
>            Reporter: Araceli Henley
>
> For example, in this macro, TestDriverPig.countStores will only count the 
> number of stores in the "test" macro, not the number of times store is 
> invoked. 
> test (in, out, column, filter_value ) returns b {
>    a = load '$in' as (name: chararray, age: int, gpa: float);
>    $b = filter a by $column < $filter_value ;
>    store $b into '$out';
> }
> x = test( '/user/hadoopqa/pignightly/tests/data/singlefile/studenttab10k', 
> '/user/hadoopqa/pignightly/out/hadoopqa.1336171525/Y_Macro_Misc_7.out.1', 
> 'age', 22 );
> x = test( '/user/hadoopqa/pignightly/tests/data/singlefile/studenttab10k', 
> '/user/hadoopqa/pignightly/out/hadoopqa.1336171525/Y_Macro_Misc_7.out.2', 
> 'gpa', 3.0 );
> There's no easy work around.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to