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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0023880
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 09. Financial managementmajorhave not tried2013-05-22 11:342015-02-25 18:20
ReportershuehnerView Statuspublic 
Assigned Toeduardo_Argal 
PrioritynormalResolutionopenFixed in Version
StatusfeedbackFix 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

0023880: FIN_AddPayment.calculateOverdueAmount is very slow

DescriptionThat 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 ReproduceI.e. using the remittance module
Use the settle/protest window
and settle many entries at once.
Proposed SolutionSee description and attached prototype patch

NOTE: that patch introduced a behavior/api-change which has to be discussed /decided upon before implementing this change.
TagsNo tags attached.
Attached Filesdiff file icon 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
Powered by Mantis Bugtracker