Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0054168
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] A. Platformmajoralways2023-12-19 18:062024-02-12 16:17
ReportermalsasuaView Statuspublic 
Assigned ToTriage Platform Conn 
PriorityhighResolutionopenFixed in Version
StatusnewFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0054168: module org.openbravo.mobile.core.transformjrxmltopdf/ should be adapted to jasper cache

DescriptionIn the issue https://issues.openbravo.com/view.php?id=37741 [^] was implemented a functionality to cache the jasper files.

the module org.openbravo.mobile.core.transformjrxmltopdf is not using this jasper cache and it could cause memory problems:

https://gitlab.com/openbravo/customers/SHARED/org.openbravo.mobile.core.transformjrxmltopdf/-/blob/master/src/org/openbravo/mobile/core/transformjrxmltopdf/JasperParser.java#L294 [^]

SHU notes about this topic:

We 'discovered' that repeated compilation + classloading of i.e. jrxml template
  Causes quite some metaspace usage (apart from being slow)

Where in backoffice we have code to cache the compiled templates (in memory) to avoid doing that repeatedly (while tomcat is running)

Doing that the below case came in in this module which was never contributed:
https://gitlab.com/openbravo/customers/SHARED/org.openbravo.mobile.core.transformjrxmltopdf/-/blob/master/src/org/openbravo/mobile/core/transformjrxmltopdf/JasperParser.java#L291 [^]

Looking at this again:
- Line 291 does NOT use backoffice jasper-cache
- Line 308 seems to implement some cache in that module on its own
https://gitlab.com/openbravo/customers/SHARED/org.openbravo.mobile.core.transformjrxmltopdf/-/blob/master/src/org/openbravo/mobile/core/transformjrxmltopdf/JasperParser.java#L308 [^]

But looking at the usage of this: here in line 216
https://gitlab.com/openbravo/customers/SHARED/org.openbravo.mobile.core.transformjrxmltopdf/-/blob/master/src/org/openbravo/mobile/core/transformjrxmltopdf/JasperParser.java#L216 [^]

Apparently it uses fillReport method passing 'String' of filename of compiled .jasper file to it
While that cache (line 308) avoids the compilation overhead is just runs that fullReport with filename over and over again


Steps To Reproducen/a
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
There are no notes attached to this issue.

- Issue History
Date Modified Username Field Change
2023-12-19 18:06 malsasua New Issue
2023-12-19 18:06 malsasua Assigned To => Triage Platform Base
2023-12-19 18:06 malsasua Modules => Core
2023-12-19 18:06 malsasua Triggers an Emergency Pack => No
2024-01-08 14:03 malsasua Description Updated View Revisions
2024-02-09 09:01 adrianromero Assigned To Triage Platform Base => Triage Platform Conn
2024-02-12 16:17 adrianromero Description Updated View Revisions


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker