[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components

Nicolas Malin (Jira)

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

Adrian Crum commented on OFBIZ-5169:
------------------------------------

Jacopo,

I agree we need to choose replacements wisely. I intended to create a Jira issue to provide some guidelines, but now that this one is here I will use it.

ArrayLists are ideal for write-once-read-many situations. Linked lists are good for read/write scenarios. If you look carefully at Mini-language, you will see where I used ArrayLists in the model object fields - because the lists are written once and then read many times. I also used the trimToSize() method to keep them small.

Linked lists are good to use in the execution path - like in local variables.

Those are general guidelines, and there will be exceptions.

               

> 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