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

Daniel Dai commented on PIG-2261:
---------------------------------

Unit test:
    all pass

test-patch:
     [exec] +1 overall.  
     [exec] 
     [exec]     +1 @author.  The patch does not contain any @author tags.
     [exec] 
     [exec]     +1 tests included.  The patch appears to include 3 new or 
modified tests.
     [exec] 
     [exec]     +1 javadoc.  The javadoc tool did not generate any warning 
messages.
     [exec] 
     [exec]     +1 javac.  The applied patch does not increase the total number 
of javac compiler warnings.
     [exec] 
     [exec]     +1 findbugs.  The patch does not introduce any new Findbugs 
warnings.
     [exec] 
     [exec]     +1 release audit.  The applied patch does not increase the 
total number of release audit warnings.

+1 for the patch.

> Restore support for parenthesis in Pig 0.9
> ------------------------------------------
>
>                 Key: PIG-2261
>                 URL: https://issues.apache.org/jira/browse/PIG-2261
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>    Affects Versions: 0.9.0
>            Reporter: Richard Ding
>            Assignee: Richard Ding
>             Fix For: 0.9.1
>
>         Attachments: PIG-2261.patch
>
>
> Pig 0.8 and earlier versions used to support syntax such as 
>  
> {code}
> A =(load ....)
> {code}
> This was removed as "useless" in 0.9 when the grammar was redone. It turns 
> out that some user is using this for ease of code generation so we want to 
> restore it back.
> Just to clarify, Pig 0.9 continues to support composite statements such as
> {code}
> B = filter (load 'data' as (a, b)) by a > 0;
> {code}
> It just removed "useless" parenthesis and doesn't support statements like
> {code}
> A = (load 'data' as (a, b));
> {code}
>  

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to