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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0027943
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajorhave not tried2014-10-22 10:232014-12-10 13:38
ReporteregoitzView Statuspublic 
Assigned Tomtaal 
PriorityimmediateResolutionfixedFixed in VersionRR15Q1
StatusclosedFix in branchpiFixed in SCM revision72727143c4da
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customerOBPS
Support ticket29824
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0027943: The OrderLoader process produce locks on the ad_sequence table.

DescriptionThe OrderLoader process produce locks on the ad_sequence table.
With high concurrency the locks can affect the performance.
Steps To Reproduce-Use simultanously several pos terminal with the same pos terminal type.
- As they use the same documenttype locks happens if the process sync is done at the same time.
*It can happen if the devides are offline for a while and all of them go back online at the same time. ON that moment all the pos are synchronized.
Proposed Solution-Take the documentno at the end of the process.
Similar than what was done on the ticket:
https://issues.openbravo.com/view.php?id=27638 [^]
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to defect 0027638 closedmarvintm The cashup process takes long in some cases and causes locks on the database 

-  Notes
(0072168)
mtaal (viewer)
2014-11-30 23:54

 After researching there are 2 documentnos retrieval which can be postponed to later in the transaction: invoice and shipment.
The order doc no is already set when it reaches the server. The payment plan document number must be set as when the payment plan is done triggers are enabled and the document no can not be set later.
(0072169)
hgbot (developer)
2014-11-30 23:55

 Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 72727143c4da1d2df1df748c7d55093bad56e3c1
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Sun Nov 30 23:55:19 2014 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/72727143c4da1d2df1df748c7d55093bad56e3c1 [^]

Fixes issue 27943: The OrderLoader process produce locks on the ad_sequence table.
Move setting of the document number as late as possible in the flow, only invoice
and shipment document number can be moved there, order doc no is already
set and payment plan document number can't be changed.

---
M src/org/openbravo/retail/posterminal/OrderLoader.java
---
(0072170)
mtaal (viewer)
2014-12-01 00:03

 I did a test using jmeter, with 30 users each syncing 10 orders simultaneously. On a fast laptop without optimization of database or other things this took 1-2 minutes in total with throughput times of each request between 30-40 seconds.

- Issue History
Date Modified Username Field Change
2014-10-22 10:23 egoitz New Issue
2014-10-22 10:23 egoitz Assigned To => marvintm
2014-10-22 10:23 egoitz OBNetwork customer => Yes
2014-10-22 10:23 egoitz Support ticket => 29824
2014-10-22 10:23 egoitz Resolution time => 1414274400
2014-10-22 10:23 egoitz Triggers an Emergency Pack => No
2014-10-22 10:23 egoitz Relationship added related to 0027638
2014-11-03 14:59 mario_castello Assigned To marvintm => mario_castello
2014-11-03 14:59 mario_castello Status new => scheduled
2014-11-03 14:59 mario_castello fix_in_branch => pi
2014-11-05 17:24 mtaal Assigned To mario_castello => mtaal
2014-11-30 23:54 mtaal Note Added: 0072168
2014-11-30 23:55 hgbot Checkin
2014-11-30 23:55 hgbot Note Added: 0072169
2014-11-30 23:55 hgbot Status scheduled => resolved
2014-11-30 23:55 hgbot Resolution open => fixed
2014-11-30 23:55 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/72727143c4da1d2df1df748c7d55093bad56e3c1 [^]
2014-12-01 00:03 mtaal Note Added: 0072170
2014-12-10 13:38 marvintm Review Assigned To => marvintm
2014-12-10 13:38 marvintm Status resolved => closed
2014-12-10 13:38 marvintm Fixed in Version => RR15Q1

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker