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

Alan Gates commented on PIG-161:
--------------------------------

Two more comments on incr5.patch:

1)  Comments on POSplit were good, but I didn't see the implementation.  I'm 
assuming this isn't finished.

2) Concerning POCast, the LoadFunc interface has been changed to provide 
conversions for data types.  This is because we can't assume that the data is 
always an ascii representation.  So POCast needs to invoke those LoadFunc calls 
rather than do the translation itself.  This implies that LOCast will need to 
contain information pointing to the correct LoadFunc so that POCast knows which 
LoadFunc to instantiate.

> 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, incr2.patch, incr3.patch, 
> incr4.patch, incr5.patch, MRCompilerTests_PlansAndOutputs.txt, 
> Phy_AbsClass.patch, physicalOps.patch, podistinct.patch, pogenerate.patch, 
> pogenerate.patch, pogenerate.patch, posort.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