Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0015957Openbravo ERPA. Platformpublic2011-01-21 14:382011-02-22 19:03
Reporteregoitz 
Assigned Toshuehner 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version2.50MP25 
Target Version2.50MP27Fixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customerOBPS
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0015957: The locator selector takes long to return the data when there are a lot of locators on the table
DescriptionThe locator selector takes long to return the data when there are a lot of locators on the table, for example with 9000 locators.
Steps To Reproduce-Create 9000 locators. you can use the attached function
-Go to Goods receipt lines.
-Open de selector locator
* it takes long to show the result
Proposed Solutionit seems that the following column on the select is not used.
REPLACE(AD_COLUMN_IDENTIFIER(TO_CHAR('M_LOCATOR'), TO_CHAR(m.M_LOCATOR_ID), TO_CHAR(?)), '''', chr(92)||'''') AS VALUE_HIDDEN

Maybe it is possible to remove it.
You will need also to remove the language parameter and the parameter on the java where the select is used.

Maybe an index could be created on the table.
Additional Information
TagsNo tags attached.
Relationships
blocks defect 00157312.50MP27 closed shuehner The locator selector takes long to return the data when there are a lot of locators on the table 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2011-02-11 17:35shuehnerTypedefect => backport
2011-02-11 17:35shuehnerfix_in_branch => 2.50
2011-02-14 12:07hgbotCheckin
2011-02-14 12:07hgbotNote Added: 0034170
2011-02-14 12:07hgbotStatusscheduled => resolved
2011-02-14 12:07hgbotResolutionopen => fixed
2011-02-14 12:07hgbotFixed in SCM revision => http://code.openbravo.com/erp/stable/2.50/rev/06ab671d69690de8997b94141346190da07615b9 [^]
2011-02-14 12:07hgbotCheckin
2011-02-14 12:07hgbotNote Added: 0034171
2011-02-14 12:07hgbotCheckin
2011-02-14 12:07hgbotNote Added: 0034172
2011-02-22 19:03marvintmNote Added: 0034325
2011-02-22 19:03marvintmStatusresolved => closed

Notes
(0034170)
hgbot   
2011-02-14 12:07   
Repository: erp/stable/2.50
Changeset: 06ab671d69690de8997b94141346190da07615b9
Author: Stefan Hühner <stefan.huehner <at> openbravo.com>
Date: Fri Feb 11 18:50:22 2011 +0100
URL: http://code.openbravo.com/erp/stable/2.50/rev/06ab671d69690de8997b94141346190da07615b9 [^]

Fixed 15957. Fix performance issue in locator selector.
Remove a column which has an expensive function call from the select, as it was
just not used.

---
M src/org/openbravo/erpCommon/info/Locator.java
M src/org/openbravo/erpCommon/info/Locator_data.xsql
---
(0034171)
hgbot   
2011-02-14 12:07   
Repository: erp/stable/2.50
Changeset: 583d55bb232dfe3958814be932e838ab49021f61
Author: Stefan Hühner <stefan.huehner <at> openbravo.com>
Date: Fri Feb 11 18:50:41 2011 +0100
URL: http://code.openbravo.com/erp/stable/2.50/rev/583d55bb232dfe3958814be932e838ab49021f61 [^]

Related to issue 15957. Reformat file

---
M src/org/openbravo/erpCommon/info/Project.java
---
(0034172)
hgbot   
2011-02-14 12:07   
Repository: erp/stable/2.50
Changeset: 60395252436151cc024a9033b110a9016b905074
Author: Stefan Hühner <stefan.huehner <at> openbravo.com>
Date: Fri Feb 11 18:55:08 2011 +0100
URL: http://code.openbravo.com/erp/stable/2.50/rev/60395252436151cc024a9033b110a9016b905074 [^]

Related to issue 15957.Do some cleanup.
- Locator.java:
  - remove some unused code/comments
  - convert use of printStacktrace to log4j
- Project.java
  - remove some unused code/comments
  - convert use of printStacktrace to log4j
- Project_java.xsql
  - remove to resultset columns which are not used

---
M src/org/openbravo/erpCommon/info/Locator.java
M src/org/openbravo/erpCommon/info/Project.java
M src/org/openbravo/erpCommon/info/Project_data.xsql
---
(0034325)
marvintm   
2011-02-22 19:03   
Verified