[
https://issues.apache.org/jira/browse/OFBIZ-5170?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13630953#comment-13630953 ]
Adrian Crum commented on OFBIZ-5170:
------------------------------------
When I overhauled Mini-language a year ago, I partially removed Javolution from it. I replaced the Javolution instances in the model fields - the reason being there is no point in borrowing an object you never intend to return. I kept the Javolution classes in the script execution path - because the borrowed objects will be returned when script execution is finished.
In revision 1465427 I replaced all instances of Javolution in Mini-language. At first I replaced all FastList instances with java.util.ArrayList. Profiling the change indicated longer AST. Then I replaced all FastList instances with java.util.LinkedList. Profiling the change indicated a shorter AST.
With Javolution
---------------
SimpleMethod.exec(MethodContext) 0.021710067 mS AST
Without Javolution
------------------
SimpleMethod.exec(MethodContext) 0.006510974 mS AST
> Profiling OFBiz
> ---------------
>
> Key: OFBIZ-5170
> URL:
https://issues.apache.org/jira/browse/OFBIZ-5170> Project: OFBiz
> Issue Type: Improvement
> Components: ALL COMPONENTS
> Affects Versions: SVN trunk
> Reporter: Adrian Crum
> Priority: Minor
> Attachments: HotSpotSettings.PNG, HotSpots_Rev_1464779.csv
>
>
> I will be using this issue to share the things I learn while profiling OFBiz to study the impact of removing Javolution.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:
http://www.atlassian.com/software/jira
Locked
