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

Anonymous | Login

Project:

News | My View | View Issues | Roadmap | Summary

View Revisions: Issue #54168		[ All Revisions ] [ Back to Issue ]
Summary	0054168: module org.openbravo.mobile.core.transformjrxmltopdf/ should be adapted to jasper cache

Revision	2024-02-12 16:17 by adrianromero
Description	In 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

Revision	2024-01-08 14:03 by malsasua
Description	In 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 [^]

Revision	2023-12-19 18:26 by malsasua
Description	the module org.openbravo.mobile.core.transformjrxmltopdf is not using the jasper cache: https://gitlab.com/openbravo/customers/SHARED/org.openbravo.mobile.core.transformjrxmltopdf/-/blob/master/src/org/openbravo/mobile/core/transformjrxmltopdf/JasperParser.java#L294 [^]