[
https://issues.apache.org/jira/browse/PIG-3199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13637373#comment-13637373
]
Cheolsoo Park commented on PIG-3199:
------------------------------------
The patch seems fine to me except the following line:
{code}
+ assertTrue("Source must end with input.txt",
lData.getSinks().get(0).endsWith("output"));
{code}
It should say "Sink must end with output" instead.
Nevertheless, I will wait for more feedback from other committers given this is
quite controversial.
> Expose LogicalPlan via PigServer API
> ------------------------------------
>
> Key: PIG-3199
> URL: https://issues.apache.org/jira/browse/PIG-3199
> Project: Pig
> Issue Type: Improvement
> Components: impl
> Affects Versions: 0.10.0
> Reporter: Prashant Kommireddi
> Assignee: Prashant Kommireddi
> Fix For: 0.12
>
> Attachments: PIG-3199_2.patch, PIG-3199.patch
>
>
> LogicalPlan could be exposed to user in order for one to make validations
> based on it. For eg, one could get Load/Store paths or other operators and be
> able to perform checks such as whether I/O paths are valid etc.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
