[ 
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

Reply via email to