You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by "Jacques Le Roux (JIRA)" <ji...@apache.org> on 2011/05/06 10:56:03 UTC
[jira] [Issue Comment Edited] (OFBIZ-4268) ResourceBundleMapWrapper
memory usage improvement
[ https://issues.apache.org/jira/browse/OFBIZ-4268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13029828#comment-13029828 ]
Jacques Le Roux edited comment on OFBIZ-4268 at 5/6/11 8:54 AM:
----------------------------------------------------------------
Thanks Philippe,
Your patch is in trunk at r1100138
Please remember to use spaces in your patches and not tabs (formatting and comparaison issues).
I finally removed a duplicated snippet (no harms just useless)
was (Author: jacques.le.roux):
Thanks Philippe,
Your patch is in trunk at r1100130
Please remember to use spaces in your patches and not tabs (formatting and comparaison issues)
> ResourceBundleMapWrapper memory usage improvement
> -------------------------------------------------
>
> Key: OFBIZ-4268
> URL: https://issues.apache.org/jira/browse/OFBIZ-4268
> Project: OFBiz
> Issue Type: Improvement
> Components: framework
> Affects Versions: SVN trunk
> Reporter: Philippe Mouawad
> Assignee: Jacques Le Roux
> Labels: MEMORY, PERFORMANCE
> Fix For: SVN trunk
>
> Attachments: patch-OFBIZ-4268.patch
>
>
> Hello,
> We are working on performance improvements while load testing the application, and I noticed that ResourceBundleMapWrapper uses arounbd 140 Ko per Serving thread, in our configuration with around 1000 thread, this is a big memory impact.
> I started investigating the class and code seems strange to me, in fact I don't understand why InternalRbmWrapper constructor copies creates a Map by copying content of ResourceBundle.
> Why not only have a reference and delegate calls to the ResourceBundle ?
> Furthermore since put methods throw RuntimeException, I really don't see any case where the Map will change.
> I made a test removing this code and delegating to resourceBundle and it seems OK.
> If someone knows why it was made like that I would really be interested, if there is no reason I will submit a patch
> Thank you
> Regards
> Philippe
> http://www.ubik-ingenierie.com
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira