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

Philippe Mouawad commented on OFBIZ-4268:
-----------------------------------------

Hello,
Thank you for your very rapid response.
I propose to implement the following:
- I will just lazy build the Map once it is requested but till then I will use 
ResourceBundle, so this will not change the behaviour, It will make us gain CPU 
+ Memory when methods not using map will be accessed.

My first tests seem OK but we plan a full UTA so if I submit this patch be 
confident It will work.

Regards
Philippe
http://www.ubik-ingenierie.com

> 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
>              Labels: MEMORY, PERFORMANCE
>
> 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

Reply via email to