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

Travis Woodruff commented on PIG-3654:
--------------------------------------

Daniel, I can't think of a reason for setting classCacheEnabled to false. I 
added it primarily so I could add the enabled vs disabled test. If we're 
satisfied with the approach, I can remove the flag.

Rohini, do you mean an assert on absolute max time (e.g., 500 ms)? I thought 
about adding that, but I was worried that load on the build server might cause 
it to fail in the absence of an actual performance regression. Happy to add it 
if you don't have that concern.


> Add class cache to PigContext
> -----------------------------
>
>                 Key: PIG-3654
>                 URL: https://issues.apache.org/jira/browse/PIG-3654
>             Project: Pig
>          Issue Type: Improvement
>    Affects Versions: 0.12.0
>            Reporter: Travis Woodruff
>            Assignee: Travis Woodruff
>            Priority: Minor
>         Attachments: PIG-3654.patch
>
>
> While benchmarking compile time for a large scripts, I noticed that class 
> lookups for UDFs accounted for a significant amount of time. Adding a simple 
> cache for looked up classes would speed up compile time noticeably.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to