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

Cheolsoo Park commented on PIG-3204:
------------------------------------

I have one very minor question. In BoundScript.java, you're not setting 
skipParseForBatch in {{registerQuery()}} while you're in {{call()}} and 
{{exec()}}. Is this because this particular method is only called by 
describe/explain/illustrate, and thus, they don't need this optimization? Or is 
there another reason why we shouldn't?

I found it inconsistent and wanted to clarify. Other than that, looks good to 
me.
                
> Reduce the number of getSchema calls during script parsing
> ----------------------------------------------------------
>
>                 Key: PIG-3204
>                 URL: https://issues.apache.org/jira/browse/PIG-3204
>             Project: Pig
>          Issue Type: Improvement
>    Affects Versions: 0.10.1
>            Reporter: Rohini Palaniswamy
>            Assignee: Rohini Palaniswamy
>             Fix For: 0.12
>
>         Attachments: PIG-3204-1.patch, PIG-3204-2.patch
>
>
>   Currently there are a lot of NN calls made to determine if there is a 
> schema file for a path in a LOAD statement. When there is a slow NN(caused by 
> whole bunch of other issues), it takes a lot of time for this and we found 
> the scripts spending anywhere from 5 mins to 40 mins depending upon the 
> script. It seems to be a good place for optimization. 

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

Reply via email to