[ 
https://issues.apache.org/jira/browse/HADOOP-1017?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12472918
 ] 

Ron Bodkin commented on HADOOP-1017:
------------------------------------

I'd be glad to remove the public API: I certainly appreciate the desire to 
avoid that complexity, or let me know if you'd rather do so. If I do so, I'll 
make the additional methods package default access to allow the tests.

> Optimization: Reduce Overhead from ReflectionUtils.newInstance
> --------------------------------------------------------------
>
>                 Key: HADOOP-1017
>                 URL: https://issues.apache.org/jira/browse/HADOOP-1017
>             Project: Hadoop
>          Issue Type: Improvement
>          Components: util
>            Reporter: Ron Bodkin
>         Attachments: ReflectionUtils.patch.txt, TestReflectionUtils.java
>
>
> I found that a significant amount of time on my project was being spent in 
> creating constructors for each row of data. I dramatically optimized this 
> performance by creating a simple WeakHashMap to cache constructors by class. 
> For example, in a sample job I find that ReflectionUtils.newInstance takes 
> 200 ms (2% of total) with the cache enabled, but it uses 900 ms (6% of total) 
> without the cache.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to