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

Santhosh Srinivasan updated PIG-294:
------------------------------------

    Attachment: boolean_test.patch

The parser works as expected. The test case was incorrect. Instead of using 
'gte' for greater than or equal to, the script was using 'ge' ala perl. I have 
added a positive and  negative test case to check if the parser works.

Patch: boolean_test.patch

The unit tests that still fail are:

    [junit] Running org.apache.pig.test.TestEvalPipeline
    [junit] Tests run: 8, Failures: 0, Errors: 1, Time elapsed: 201.12 sec
    [junit] Test org.apache.pig.test.TestEvalPipeline FAILED

    [junit] Running org.apache.pig.test.TestFilterOpNumeric
    [junit] Tests run: 8, Failures: 0, Errors: 1, Time elapsed: 55.618 sec
    [junit] Test org.apache.pig.test.TestFilterOpNumeric FAILED

    [junit] Running org.apache.pig.test.TestStoreOld
    [junit] Tests run: 3, Failures: 0, Errors: 2, Time elapsed: 39.291 sec
    [junit] Test org.apache.pig.test.TestStoreOld FAILED


> Parse errors for boolean conditions
> -----------------------------------
>
>                 Key: PIG-294
>                 URL: https://issues.apache.org/jira/browse/PIG-294
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>    Affects Versions: types_branch
>            Reporter: Santhosh Srinivasan
>            Assignee: Santhosh Srinivasan
>         Attachments: boolean_test.patch
>
>
> The parser throws exceptions for pig statements that contain boolean 
> conditions with operands that use string comparators. A sample statement to 
> reproduce the test is given below:
> split a into b if name lt 'f', c if (name ge 'f' and name le 'h'), d if name 
> gt 'h';

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