Activity

analysis shows that 98% of trait maps contain 3 entries with remainder containing 1 entry; changing to tree map results in a 2% memory savings compared to overall usage by java.util.* classes at IFDocumentHandler.endPage() call time; time performance also improved (marginally) with tree map, so there are no negatives in making this change

Glenn Adams
added a comment - 06/Apr/12 19:12 analysis shows that 98% of trait maps contain 3 entries with remainder containing 1 entry; changing to tree map results in a 2% memory savings compared to overall usage by java.util.* classes at IFDocumentHandler.endPage() call time; time performance also improved (marginally) with tree map, so there are no negatives in making this change
see http://svn.apache.org/viewvc?view=revision&revision=1310524

on further consideration, Sergey, if you could provide some data that shows relative gain/loss in heap use and processing time for an large FO file of 'typical' complexity when using the current code HashMap(20) versus TreeMap, then I would be inclined to make this change (assuming the data favors TreeMap);

Glenn Adams
added a comment - 05/Apr/12 06:04 on further consideration, Sergey, if you could provide some data that shows relative gain/loss in heap use and processing time for an large FO file of 'typical' complexity when using the current code HashMap(20) versus TreeMap, then I would be inclined to make this change (assuming the data favors TreeMap);