[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13623393#comment-13623393 ]
Jacopo Cappellato commented on OFBIZ-5169: ------------------------------------------ Using ArrayList, rather than LinkedList, as a default is a good approach because ArrayList will work better in most situations (in terms of algorithmic runtimes and memory footprints). However I think it is important to wisely choose ArrayList or LinkedList based on the context, because sometimes a linked list may be a better choice: I am not saying we should do it now and we can tackle this at a later moment, possibly focusing only on list that can be very big and analyzing their usage to see if array or linked list is better; it would be also nice, when we use ArrayList, to see if we can predict the capacity of the list and allocate it at creation: if we can do this we will have a more predictable memory usage and less unpredictable peaks under heavy load (when the array capacity is reached and the array is copied to a new one with double capacity). > Removing Javolution from framework components > --------------------------------------------- > > Key: OFBIZ-5169 > URL: https://issues.apache.org/jira/browse/OFBIZ-5169 > Project: OFBiz > Issue Type: Improvement > Components: framework > Affects Versions: SVN trunk > Reporter: Varun Bhansaly > Priority: Minor > Fix For: SVN trunk > > Attachments: framework-FastList-removed.patch > > > a. Remove static instances of Javolution objects. > b. Remove other uses of Javolution objects. -- 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