[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15029677#comment-15029677 ] Taher Alkhateeb commented on OFBIZ-5169: I've investigated what to re implement to satisfy dependencies in other Java sources in the framework and I list them below for reference. Once we reimplement them in EntitySaxReader without javolution then I would assume it is safe to remove the javolution from the framework EntitySaxReader(delegator, timeout) // constructor EntitySaxReader(delegator) // constructor parse(content) // XML parse(location) // URL setCheckDataOnly(checkDataOnly) // boolean setCreateDummyFks(dummyFks) // boolean setMaintainTxStamps(maintainTxStamps) // boolean setTransactionTimeout(transactionTimeout) // int setUseTryInsertMethod(value) // boolean > 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: Trunk >Reporter: Varun Bhansaly >Priority: Minor > Attachments: framework-FastList-removed.patch > > > a. Remove static instances of Javolution objects. > b. Remove other uses of Javolution objects. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14974930#comment-14974930 ] Nicolas Malin commented on OFBIZ-5169: -- greats! thanks Taher > 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: Trunk >Reporter: Varun Bhansaly >Priority: Minor > Attachments: framework-FastList-removed.patch > > > a. Remove static instances of Javolution objects. > b. Remove other uses of Javolution objects. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14970491#comment-14970491 ] Taher Alkhateeb commented on OFBIZ-5169: I removed all javolution uses except in EntitySaxReader.java in r1710122. Now one file remains and we can take this library out. Working on it. > 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: Trunk >Reporter: Varun Bhansaly >Priority: Minor > Attachments: framework-FastList-removed.patch > > > a. Remove static instances of Javolution objects. > b. Remove other uses of Javolution objects. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14510548#comment-14510548 ] Deepak Dixit commented on OFBIZ-5169: - We should use the LinkedHashMap for controller pre/post processor eventLists (ConfigXMLReader.java:180) as HashMap does not maintain the iteration order. 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: Trunk Reporter: Varun Bhansaly Priority: Minor Attachments: framework-FastList-removed.patch a. Remove static instances of Javolution objects. b. Remove other uses of Javolution objects. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14510628#comment-14510628 ] Nicolas Malin commented on OFBIZ-5169: -- Thanks Jacopo I appreciate to have an other opinion. I open the issue OFBIZ-6298 for that 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: Trunk Reporter: Varun Bhansaly Priority: Minor Attachments: framework-FastList-removed.patch a. Remove static instances of Javolution objects. b. Remove other uses of Javolution objects. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14511170#comment-14511170 ] Martin Becker commented on OFBIZ-5169: -- Well, my intention is to overall switch the now used LinkedList to ArrayList, as it is the more proper default with a light performance drawback in rare cases, if at all, and to be in line with the types used in groovy scripts. No functional drawback compared to FastList is to be expected (in contrast to HashMap/HashSet, if preserving the insert-order for iteration is a requirement). That would be a good starting point for detailed analysis and changes in the scope of [OFBIZ-6298], to optimize the use of collection types, also this will be a very exhausting activity over thousands of code places with very small expectable benefits in real world performance. 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: Trunk Reporter: Varun Bhansaly Priority: Minor Attachments: framework-FastList-removed.patch a. Remove static instances of Javolution objects. b. Remove other uses of Javolution objects. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14509884#comment-14509884 ] Nicolas Malin commented on OFBIZ-5169: -- I also agree with you (really it's not a joke) I started the conversion on OFBIZ-5781 where Deepak Dixit began by groovy file. When I kept, I didn't read this thread so I checked what is more used on framework. Now I understand more each detail and I have two choices : * revert and restart the conversion (lot's of work and javolution come back) * keep like it and realize a second pass with the ArrayList improvement I really hesitated between both to finally says, ok I break nothing, I didn't impacted the framework, so move forward. And sure, it will be a pleasure for me to review your future patch ;) 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: Trunk Reporter: Varun Bhansaly Priority: Minor Attachments: framework-FastList-removed.patch a. Remove static instances of Javolution objects. b. Remove other uses of Javolution objects. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14510494#comment-14510494 ] Jacopo Cappellato commented on OFBIZ-5169: -- Thank you Nicolas for your feedback. I think that a second pass would make more sense, we could file a ticket in Jira and then see if volunteers will help us. 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: Trunk Reporter: Varun Bhansaly Priority: Minor Attachments: framework-FastList-removed.patch a. Remove static instances of Javolution objects. b. Remove other uses of Javolution objects. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14506776#comment-14506776 ] Jacopo Cappellato commented on OFBIZ-5169: -- I agree with [~mbecker] 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: Trunk Reporter: Varun Bhansaly Priority: Minor Attachments: framework-FastList-removed.patch a. Remove static instances of Javolution objects. b. Remove other uses of Javolution objects. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14494135#comment-14494135 ] Martin Becker commented on OFBIZ-5169: -- I wonder why the LinkedList was chosen as the default replacement of FastList in Java code instead if an ArrayList (see comment from Jacopo Cappellato above). Although it could be a very complex question which implementation to choose respecting the concrete logic and the underlying environment, I would prefer to use ArrayList as default on the basis of the memory consumption and general performance impacts of LinkedList alone. The cases at OFBiz where the main benefit of faster insert/remove in the middle/beginning of a large LinkedList is relevant should be very rare, especially with moderate list sizes. The disadvantage of ArrayList in growing over the initial capacity should always be encountered by specifying an appropriate initial capacity where possible. See short description from oracle: https://docs.oracle.com/javase/tutorial/collections/implementations/list.html Another conspicuity: The migrated groovy scripts instead often using the default list implementation which is ArrayList by only using „[]“ without specifying LinkedList as concrete type. 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: Trunk Reporter: Varun Bhansaly Priority: Minor Attachments: framework-FastList-removed.patch a. Remove static instances of Javolution objects. b. Remove other uses of Javolution objects. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[
https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14490907#comment-14490907
]
Jacques Le Roux commented on OFBIZ-5169:
Yes maybe a profiler session would be a plus. But I have no ideas on how much
OFBIZ-5781 changes performances. I note here Adrian comment on dev ML about
OFBIZ-5781 and r1672752
{quote}
This should be fine, but there might be cases where a LinkedHashMap is needed.
{quote}
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: Trunk
Reporter: Varun Bhansaly
Priority: Minor
Attachments: framework-FastList-removed.patch
a. Remove static instances of Javolution objects.
b. Remove other uses of Javolution objects.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14490932#comment-14490932 ] Nicolas Malin commented on OFBIZ-5169: -- Yesterday I continue to benchmark. On the ant clean-all load-demo run-tests between before and after the revision r1672752 on average I win 10s. If some body can to do it under an other plateform (I'm under linux and jdk 1.7), it's would be great :) For the LinkedHashMap, yes I use it when I detected an iteration on the map. I will realize a second pass to check if I can found other optimization, it's also the case for arrayList instead of linkedList. On many case the list use through foreach loop, but I didn't analyze all case. 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: Trunk Reporter: Varun Bhansaly Priority: Minor Attachments: framework-FastList-removed.patch a. Remove static instances of Javolution objects. b. Remove other uses of Javolution objects. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[
https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14490310#comment-14490310
]
Nicolas Malin commented on OFBIZ-5169:
--
After close the OFBIZ-5781 we still face in code
framework/entity/src/org/ofbiz/entity/util/EntitySaxReader.java:
{code}
import javolution.text.CharArray;
import javolution.text.Text;
import javolution.xml.sax.Attributes;
import javolution.xml.sax.XMLReaderImpl;
public class EntitySaxReader implements javolution.xml.sax.ContentHandler,
ErrorHandler {
{code}
I will check how to replace it.
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: Trunk
Reporter: Varun Bhansaly
Priority: Minor
Attachments: framework-FastList-removed.patch
a. Remove static instances of Javolution objects.
b. Remove other uses of Javolution objects.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14490526#comment-14490526 ] Nicolas Malin commented on OFBIZ-5169: -- I re read this thread and I am afraid that my commit to close OFBIZ-5781 has been untimely. Do you want that I revert it ? 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: Trunk Reporter: Varun Bhansaly Priority: Minor Attachments: framework-FastList-removed.patch a. Remove static instances of Javolution objects. b. Remove other uses of Javolution objects. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13625245#comment-13625245 ] Adrian Crum commented on OFBIZ-5169: Thinking about this more... It would be best if we worked on the components in this order: base, entity, service. I will run the profiler as each component is changed. When all three components have been changed, and if the profiler shows no adverse performance impact, then I believe we can remove Javolution from the rest of the project without concern. 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
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13624976#comment-13624976 ] Adrian Crum commented on OFBIZ-5169: Varun, It would be easier for me to evaluate the changes if we replace all Javolution objects, instead of just Lists. Also, could you break up the patches by component please? 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
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13625094#comment-13625094 ] Varun Bhansaly commented on OFBIZ-5169: --- Adrian, Removing all Javolution objects would have taken more time hence had replaced only the lists. I will try to break up the patch into smaller patches by component. 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
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=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
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13623419#comment-13623419 ] Adrian Crum commented on OFBIZ-5169: Varun, Thank you for getting started on this. 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
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=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
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13623543#comment-13623543 ] Varun Bhansaly commented on OFBIZ-5169: --- Adrian, Will you will committing this patch in the branch and shall I move ahead with removing FastSet FastMap ? Anything specific you would like to be considered while creating patch for these ? FastMap changes may have to be handled relatively carefully. 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
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13623545#comment-13623545 ] Adrian Crum commented on OFBIZ-5169: I will be committing the changes to the trunk. I came to the conclusion a branch will be too hard to maintain because of the scope of the effort. I will need time to review your patch and profile the changes to check for negative impact. 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
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13623553#comment-13623553 ] Jacopo Cappellato commented on OFBIZ-5169: -- Adrian, I think that gathering information from a profiler would indeed help. However, would it make sense, if there is a general agreement, to create a branch with the current trunk to keep a Javolution version of OFBiz and then commit the changes to remove Javolution (after your review/tests) to the trunk? I am basically saying that we could use the branch the other way round: in the future we could use it to run comparisons with the Javolution-free trunk. A TAG would also probably work (because the Javolution branch will be mostly read-only) but a branch may give us some flexibility: for example we could commit there some optimizations done in the trunk that are not related to Javolution and that may impact performance. In this way it will be easier to migrate out of Javolution in steps. Of course before doing this we should check with the community if there are still major concerns. 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
[jira] [Commented] (OFBIZ-5169) Removing Javolution from framework components
[ https://issues.apache.org/jira/browse/OFBIZ-5169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13623566#comment-13623566 ] Adrian Crum commented on OFBIZ-5169: That's a great idea! I can use the one I already created and revert my changes (only one commit so far). 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
