Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0036589Retail ModulesWeb POSpublic2017-08-03 11:492017-08-08 13:17
Reporterjonibc 
Assigned Toranjith_qualiantech_com 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR17Q4 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customerOBPS
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0036589: [SERQA 3180] Performance problem getting documentNo
DescriptionHaving BUT Production Environment, if more than one order is created using Ecommerce (External Order Loader), the same documentNo is assigned to all the orders created at the same time.

To avoid this behavior the following line was added:

OBDal.getInstance().refresh(lockedTerminal);

But it takes around a second to refresh the entity.
Steps To Reproduce*Install BUT Production environment
*Using Jmeter, configure to send 3 ecommerce orders at the same time
*Using Java Profiler, check the time needed by the refresh method.
*Check in the backend that the same documentNo is assigned for the 3 orders
Proposed SolutionEvict the lockedTerminal to clean it in cache.
Do get() to obtain the right object.

Attached diff with the solution.
Additional Information
TagsNo tags attached.
Relationships
caused by defect 0036423 closed ranjith_qualiantech_com [SERQA 3088] Duplicated documentNo for orders created in the same moment using External Order Loader 
Attached Filesdiff posTerminal.diff (870) 2017-08-03 11:49
https://issues.openbravo.com/file_download.php?file_id=10959&type=bug
Issue History
Date ModifiedUsernameFieldChange
2017-08-03 11:49jonibcNew Issue
2017-08-03 11:49jonibcAssigned To => Retail
2017-08-03 11:49jonibcFile Added: posTerminal.diff
2017-08-03 11:49jonibcOBNetwork customer => Yes
2017-08-03 11:49jonibcResolution time => 1503007200
2017-08-03 11:49jonibcTriggers an Emergency Pack => No
2017-08-03 11:50jonibcRelationship addedcaused by 0036423
2017-08-03 11:50jonibcIssue Monitored: jonibc
2017-08-03 14:27ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2017-08-03 14:28ranjith_qualiantech_comStatusnew => scheduled
2017-08-04 08:00hgbotCheckin
2017-08-04 08:00hgbotNote Added: 0098378
2017-08-04 08:00hgbotStatusscheduled => resolved
2017-08-04 08:00hgbotResolutionopen => fixed
2017-08-04 08:00hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/e7e610b702c92365fd49d3a682822fde637ea25d [^]
2017-08-08 13:17marvintmReview Assigned To => marvintm
2017-08-08 13:17marvintmStatusresolved => closed
2017-08-08 13:17marvintmFixed in Version => RR17Q4

Notes
(0098378)
hgbot   
2017-08-04 08:00   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: e7e610b702c92365fd49d3a682822fde637ea25d
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Fri Aug 04 11:29:55 2017 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/e7e610b702c92365fd49d3a682822fde637ea25d [^]

Fixed issue 36589 : Terminal should be loaded by cleaning cache in ExternalorderLoader

* Clear Terminal from cache and load the temrimal inorder to avaoid cache

---
M src/org/openbravo/retail/posterminal/ExternalOrderLoader.java
---