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

Kyungho Jeon commented on PIG-3793:
-----------------------------------

Thank you for the response. I found that {{LogicalPlanData}} instance is 
created and initialized when {{PigServer.getLogicalPlanData()}} is called. 
Then, my concern becomes less significant. 

But now I am thinking that {{LogicalPlanData}} should work as an interface (in 
a general meaning, not a Java interface) to {{LogicalPlan}} rather than a 
stateful object, considering that the class was added to avoid exposing 
{{LogicalPlan}}. It won't be a problem in current Pig implementation right now. 
But if someone (like me, actually) is playing with {{LogicalPlan}} and its 
optimizers, for example {{AddForEach}}, then the fact that {{LogicalPlanData}} 
computes and keeps the value might be confusing. 

I have been following Pig for a year but I am still a newbie. Please forgive my 
pestering. I am just trying to understand the implementation. Any comment will 
be appreciated!

> Provide info on number of LogicalRelationalOperator(s) used in the script 
> through LogicalPlanData
> -------------------------------------------------------------------------------------------------
>
>                 Key: PIG-3793
>                 URL: https://issues.apache.org/jira/browse/PIG-3793
>             Project: Pig
>          Issue Type: Improvement
>          Components: data
>    Affects Versions: 0.13.0
>            Reporter: Prashant Kommireddi
>            Assignee: Prashant Kommireddi
>             Fix For: 0.13.0
>
>         Attachments: PIG-3793.patch, PIG-3793_2.patch
>
>
> Its useful to have an understanding of how many operators are being used in 
> the script via the API. This could allow admins to enforce 
> checks/restrictions on the length/complexity of the plan in user scripts.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to