Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||||||
ID | ||||||||||||
0023880 | ||||||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||||||
defect | [Openbravo ERP] 09. Financial management | major | have not tried | 2013-05-22 11:34 | 2015-02-25 18:20 | |||||||
Reporter | shuehner | View Status | public | |||||||||
Assigned To | eduardo_Argal | |||||||||||
Priority | normal | Resolution | open | Fixed in Version | ||||||||
Status | feedback | 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 | |||||||||||
Review Assigned To | ||||||||||||
Web browser | ||||||||||||
Modules | Core | |||||||||||
Regression level | ||||||||||||
Regression date | ||||||||||||
Regression introduced in release | ||||||||||||
Regression introduced by commit | ||||||||||||
Triggers an Emergency Pack | No | |||||||||||
Summary | 0023880: FIN_AddPayment.calculateOverdueAmount is very slow | |||||||||||
Description | That function is very slow taking up to 600ms to process a single entry. Analyzing this slowdown shows a very high dal overhead calling the small pl-function APRM_ISPAYMENTCONFIRMED. As that function is supposd to be stable (returning same output when giving same input several times) adding a java hashmap as a java based cache does improve runtime greatly. Observed runtime improvement in one customer is from 5min30sec downto 20-25seconds. | |||||||||||
Steps To Reproduce | I.e. using the remittance module Use the settle/protest window and settle many entries at once. | |||||||||||
Proposed Solution | See description and attached prototype patch NOTE: that patch introduced a behavior/api-change which has to be discussed /decided upon before implementing this change. | |||||||||||
Tags | No tags attached. | |||||||||||
Attached Files | ispaymentconfirmed_cache.diff [^] (1,614 bytes) 2013-05-22 11:35 [Show Content] | |||||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | |
Notes | |
(0074921) eduardo_Argal (developer) 2015-02-25 18:20 |
Any idea is this is old? It seems the original code is not there |
Issue History | |||
Date Modified | Username | Field | Change |
2013-05-22 11:34 | shuehner | New Issue | |
2013-05-22 11:34 | shuehner | Assigned To | => dmiguelez |
2013-05-22 11:34 | shuehner | Modules | => Core |
2013-05-22 11:34 | shuehner | Triggers an Emergency Pack | => No |
2013-05-22 11:35 | shuehner | File Added: ispaymentconfirmed_cache.diff | |
2014-09-03 14:10 | eduardo_Argal | Assigned To | dmiguelez => eduardo_Argal |
2015-02-25 18:20 | eduardo_Argal | Note Added: 0074921 | |
2015-02-25 18:20 | eduardo_Argal | Status | new => feedback |
Copyright © 2000 - 2009 MantisBT Group |