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

Rohini Palaniswamy commented on PIG-3654:
-----------------------------------------

  You are right about slow machines taking time and causing test failures. Come 
to think of it wondering if we even need the test. Obviously caching and 
fetching from map is going to be better than doing reflection. If we are 
keeping caching default and no option to turn it off, I think we can just 
remove the test. We usually don't have a before and after or with and without 
test when we do performance fixes. You can just document the performance 
numbers here in this jira. 

> 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