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

Shubham Chopra updated PIG-161:
-------------------------------

    Attachment: arithmeticOperators.patch

Response to Alan's comments:

This patch contains the arithmetic physical operators with unit tests. I have 
done some modifications suggested by Alan regarding passing the errors along 
instead of absorbing them. I have also added the mod operator for integers and 
longs.

Also, the arithmeticOperators don't differ from the binaryOperators. The 
specific class was just to make the code easier to follow. It can be removed if 
you think its not necessary.

I am working on the POGenerate to implement the modifications you suggested.



> Rework physical plan
> --------------------
>
>                 Key: PIG-161
>                 URL: https://issues.apache.org/jira/browse/PIG-161
>             Project: Pig
>          Issue Type: Sub-task
>            Reporter: Alan Gates
>            Assignee: Alan Gates
>         Attachments: arithmeticOperators.patch, Phy_AbsClass.patch, 
> pogenerate.patch
>
>
> This bug tracks work to rework all of the physical operators as described in 
> http://wiki.apache.org/pig/PigTypesFunctionalSpec

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