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 | |||||||||||
| 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 | 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 | ||||||||||||
		  Relationships		[ Relation Graph ] 
		[ Dependency Graph ] 
			 | 
|
  		Notes	 | 
|
| 
				(0074921) eduardo_Argal (viewer) 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 | OBNetwork customer | => No | 
| 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 |