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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0048756
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 09. Financial managementmajoralways2022-03-10 07:292022-04-20 09:41
ReporteralostaleView Statuspublic 
Assigned Toalostale 
PrioritynormalResolutionfixedFixed in VersionPR22Q2
StatusclosedFix 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

0048756: unnecesary OrganizationStructureProvider double initialization on ticket creation

DescriptionAs reported in 0042370, OrganizationStructureProvider initialization has an important impact in the overall database performance.

When creating tickets from the POS this initialization occurs twice per ticket:

1. Creating the context for processing the ticket (might be addressed by 0042370)
2. When obtaining document type (the goal of this issue).
Steps To ReproduceIn a retail instance:

1. Log in as System Admin and set debug log level to OrganizationStructureProvider
2. Log in POS and create one ticket
3. Check logs, there should be a single initialization there are two:

2022-03-10 07:28:36,465 [Import Entry - 9] DEBUG org.openbravo.dal.security.OrganizationStructureProvider - Client 39363B0921BB4293B48383844325E84C initialized in 1.023 ms
2022-03-10 07:28:36,554 [Import Entry - 9] DEBUG org.openbravo.dal.security.OrganizationStructureProvider - Client 39363B0921BB4293B48383844325E84C initialized in 0.455 ms
TagsPerformance
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to defect 0042370 closedAugustoMauch Unnecessary executions of a query that loads the complete organizational structure of a client 

-  Notes
(0135608)
hgbot (developer)
2022-03-10 07:38

Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/538 [^]
(0135735)
hgbot (developer)
2022-03-15 14:46

Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/538 [^]
(0135736)
hgbot (developer)
2022-03-15 14:46

Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/openbravo [^]
Changeset: 8a9b434def6ead92f787335bf2fa88e60d6d4fdd
Author: Asier Lostalé <asier.lostale@openbravo.com>
Date: 15-03-2022 13:45:00
URL: https://gitlab.com/openbravo/product/openbravo/-/commit/8a9b434def6ead92f787335bf2fa88e60d6d4fdd [^]

fixes BUG-48756: unnecesary OSP initialization in getDocumentType

Whenever FIN_Utility.getDocumentType was invoked, OSP was initialized.
In high volume environments, this causes a noticeable DB overhead in ticket
creation.

Instead of creating a new OSP instance which needs to be initialized, we
now reuse the one from the context that is already initialized.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/utility/FIN_Utility.java
---
(0136615)
plujan (manager)
2022-04-20 09:41

Please change the generic user to the actual person assigned when fixing an issue

- Issue History
Date Modified Username Field Change
2022-03-10 07:29 alostale New Issue
2022-03-10 07:29 alostale Assigned To => Triage Omni OMS
2022-03-10 07:29 alostale Modules => Core
2022-03-10 07:29 alostale Triggers an Emergency Pack => No
2022-03-10 07:30 alostale Tag Attached: Performance
2022-03-10 07:30 alostale Relationship added related to 0042370
2022-03-10 07:34 alostale Summary unnecesary OrganizationStructureProvider initialization on ticket creation => unnecesary OrganizationStructureProvider double initialization on ticket creation
2022-03-10 07:38 hgbot Note Added: 0135608
2022-03-15 14:46 hgbot Resolution open => fixed
2022-03-15 14:46 hgbot Status new => closed
2022-03-15 14:46 hgbot Note Added: 0135735
2022-03-15 14:46 hgbot Fixed in Version => PR22Q2
2022-03-15 14:46 hgbot Note Added: 0135736
2022-04-20 09:41 plujan Note Added: 0136615
2022-04-20 09:41 plujan Assigned To Triage Omni OMS => alostale


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker