Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||||||
| ID | ||||||||||||
| 0049498 | ||||||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||||||
| defect | [Openbravo ERP] 09. Financial management | major | have not tried | 2022-06-07 17:08 | 2022-06-07 17:08 | |||||||
| Reporter | shuehner | View Status | public | |||||||||
| Assigned To | Triage Omni OMS | |||||||||||
| Priority | normal | Resolution | open | Fixed in Version | ||||||||
| Status | new | Fix in branch | Fixed in SCM revision | |||||||||
| Projection | none | ETA | none | Target Version | ||||||||
| OS | Any | Database | Any | Java version | ||||||||
| OS Version | Database version | Ant version | ||||||||||
| Product Version | SCM revision | |||||||||||
| Merge Request Status | ||||||||||||
| Review Assigned To | ||||||||||||
| OBNetwork customer | No | |||||||||||
| Web browser | ||||||||||||
| Modules | Core | |||||||||||
| Support ticket | ||||||||||||
| Regression level | ||||||||||||
| Regression date | ||||||||||||
| Regression introduced in release | ||||||||||||
| Regression introduced by commit | ||||||||||||
| Triggers an Emergency Pack | No | |||||||||||
| Summary | 0049498: General Ledger Journal Report does not properly implement xsql-scrollable interface (and uses too much memory) | |||||||||||
| Description | Issue 35157 from 2017 converted this report to use the xsql scrollable interface which avoids loading all rows of the result into an Vector or java-array at once. However the converted code is buggy and loads the scrollable output directly into Vector/java-array defeating the improvement: Example here: https://gitlab.com/openbravo/product/openbravo/-/blob/master/src/org/openbravo/erpCommon/ad_reports/ReportGeneralLedgerJournal.java#L955 [^] When being worked all scrollable-xsql calls should be reviewed. | |||||||||||
| Steps To Reproduce | https://gitlab.com/openbravo/product/openbravo/-/blob/master/src/org/openbravo/erpCommon/ad_reports/ReportGeneralLedgerJournal.java#L955 [^] | |||||||||||
| Proposed Solution | Pass scrollable interface directly into renderJR as intended. The size=0 check can be probably replaced by scrollable.hasData() call | |||||||||||
| Tags | No tags attached. | |||||||||||
| Attached Files | ||||||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
| There are no notes attached to this issue. |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2022-06-07 17:08 | shuehner | New Issue | |
| 2022-06-07 17:08 | shuehner | Assigned To | => Triage Omni OMS |
| 2022-06-07 17:08 | shuehner | OBNetwork customer | => No |
| 2022-06-07 17:08 | shuehner | Modules | => Core |
| 2022-06-07 17:08 | shuehner | Triggers an Emergency Pack | => No |
| 2022-06-07 17:09 | shuehner | Relationship added | related to 0035157 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |