[
https://issues.apache.org/jira/browse/PIG-161?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12597738#action_12597738
]
Shravan Matthur Narayanamurthy commented on PIG-161:
----------------------------------------------------
I think I know the problem...
The Union & JobSubmission use test cases of physical plans that use the dummy
POCast operator. When shubham replaced them with the actual POCast, he has
forgotten to replace the use of dummy POCast appropriately. Two ways to solve
this:
1) Either leave the dummy POCast alone by renaming it.
2) Remove them from the plans for those test cases and replace them with the
correct POCast operator.
I think 2 is better though its a bit more work
> 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, BinCondAndNegative.patch,
> CastAndMapLookUp.patch, incr2.patch, incr3.patch, incr4.patch, incr5.patch,
> logToPhyTranslator.patch, MRCompilerTests_PlansAndOutputs.txt,
> Phy_AbsClass.patch, physicalOps.patch, physicalOps.patch, physicalOps.patch,
> physicalOps.patch, physicalOps_latest.patch, POCast.patch, podistinct.patch,
> pogenerate.patch, pogenerate.patch, pogenerate.patch, posort.patch,
> TEST-org.apache.pig.test.TestLocalJobSubmission.txt,
> TEST-org.apache.pig.test.TestUnion.txt, translator.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.
