You are viewing a plain text version of this content. The canonical link for it is here.
Posted to fop-dev@xmlgraphics.apache.org by bu...@apache.org on 2011/09/14 13:11:22 UTC

DO NOT REPLY [Bug 51808] New: Minor memory waste in Area

https://issues.apache.org/bugzilla/show_bug.cgi?id=51808

             Bug #: 51808
           Summary: Minor memory waste in Area
           Product: Fop
           Version: 1.0
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: general
        AssignedTo: fop-dev@xmlgraphics.apache.org
        ReportedBy: vlsergey@gmail.com
    Classification: Unclassified


It is usual for Area::props to have 1-2 properties, not 20. From other point,
20 seems to be max cap. To prevent memory waste HashMap(20) should be replaced
with TreeMap().

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 51808] Minor memory waste in Area

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=51808

Glenn Adams <gl...@skynav.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |LATER

--- Comment #1 from Glenn Adams <gl...@skynav.com> 2012-04-04 22:15:43 UTC ---
feel free to propose a patch; in the mean time, closing due to lack of action

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 51808] Minor memory waste in Area

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=51808

Glenn Adams <gl...@skynav.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|LATER                       |

--- Comment #2 from Glenn Adams <gl...@skynav.com> 2012-04-05 05:04:37 UTC ---
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);

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 51808] Minor memory waste in Area

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=51808

Glenn Adams <gl...@skynav.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Glenn Adams <gl...@skynav.com> 2012-04-06 18:12:57 UTC ---
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

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.