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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0037653
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajorhave not tried2018-01-17 10:142019-12-09 15:00
ReporteraferrazView Statuspublic 
Assigned Toranjith_qualiantech_com 
PrioritynormalResolutionfixedFixed in VersionRR20Q1
StatusclosedFix in branchFixed in SCM revisiondcbcd845f19a
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tomarvintm
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0037653: Performance problem in PaidReceipts

DescriptionPerformance problem in PaidReceipts
Steps To ReproduceProcess takes around 50 ms.
If you call this process many many times (to generate the JSON of orders stored in database), it will take many hours:
1,000,000 records * 50 ms. = 50,000,000 ms. = 13.88 hours.

Attached yourkit snapshot.
Proposed SolutionFollowing query is taking most of the time: PaidReceipts.java:323.
Subquery inside it is almost the same as the query run just before in PaidReceipts.java:304.
It seems subquery in line 323 could be replaced by the result of query in line 304.
TagsNo tags attached.
Attached Fileshtml file icon PaidReceipts.html [^] (56,185 bytes) 2018-01-17 10:14
zip file icon PaidReceipts.snapshot.zip [^] (1,674,857 bytes) 2018-01-17 10:16
patch file icon issue 37653 source posterminal.patch [^] (2,891 bytes) 2018-03-28 08:14 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]
related to defect 0042385 closedrqueralta Performance improvement in PaidReceipts query 

-  Notes
(0103680)
hgbot (developer)
2018-04-05 07:52

Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: dcbcd845f19ac1ab4f58d76cd7365a5175dd5440
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Apr 05 11:22:21 2018 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/dcbcd845f19ac1ab4f58d76cd7365a5175dd5440 [^]

Fixed issue 37653 : Refactored PaidReceipts for Performance problem

---
M src/org/openbravo/retail/posterminal/PaidReceipts.java
---
(0103915)
marvintm (manager)
2018-04-16 18:09

There are several problems with the current solution:

- Concatenation of strings in this way is a potential problem of hql injection
- Too many values passed like this may cause a problem in Oracle
- Not clear why the change would improve the performance in this case, and the commit message doesn't explain it.
(0105453)
hgbot (developer)
2018-06-26 14:03

Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 78efe957658de849054997b779a73b9ba4ebfe47
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Tue Jun 26 17:32:51 2018 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/78efe957658de849054997b779a73b9ba4ebfe47 [^]

Related to issue 37653 : Backed out changeset: dcbcd845f19a

* Changes are not enough to fix the perfomance issues.

---
M src/org/openbravo/retail/posterminal/PaidReceipts.java
---
(0116135)
ranjith_qualiantech_com (developer)
2019-12-07 08:11

Fixed in https://issues.openbravo.com/view.php?id=42385 [^]

- Issue History
Date Modified Username Field Change
2018-01-17 10:14 aferraz New Issue
2018-01-17 10:14 aferraz Assigned To => Retail
2018-01-17 10:14 aferraz Triggers an Emergency Pack => No
2018-01-17 10:14 aferraz File Added: PaidReceipts.html
2018-01-17 10:16 aferraz File Added: PaidReceipts.snapshot.zip
2018-01-17 11:10 aferraz Steps to Reproduce Updated View Revisions
2018-01-29 08:56 aferraz Proposed Solution updated
2018-03-26 14:57 ranjith_qualiantech_com Assigned To Retail => ranjith_qualiantech_com
2018-03-26 14:57 ranjith_qualiantech_com Status new => scheduled
2018-03-28 08:14 ranjith_qualiantech_com File Added: issue 37653 source posterminal.patch
2018-04-05 07:52 hgbot Checkin
2018-04-05 07:52 hgbot Note Added: 0103680
2018-04-05 07:52 hgbot Status scheduled => resolved
2018-04-05 07:52 hgbot Resolution open => fixed
2018-04-05 07:52 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/dcbcd845f19ac1ab4f58d76cd7365a5175dd5440 [^]
2018-04-16 18:09 marvintm Note Added: 0103915
2018-04-16 18:09 marvintm Status resolved => new
2018-04-16 18:09 marvintm Resolution fixed => open
2018-04-23 13:10 ranjith_qualiantech_com Status new => scheduled
2018-06-26 14:03 hgbot Checkin
2018-06-26 14:03 hgbot Note Added: 0105453
2019-12-07 08:11 ranjith_qualiantech_com Relationship added related to 0042385
2019-12-07 08:11 ranjith_qualiantech_com Note Added: 0116135
2019-12-07 08:11 ranjith_qualiantech_com Status scheduled => resolved
2019-12-07 08:11 ranjith_qualiantech_com Resolution open => fixed
2019-12-09 15:00 marvintm Review Assigned To => marvintm
2019-12-09 15:00 marvintm Status resolved => closed
2019-12-09 15:00 marvintm Fixed in Version => RR20Q1


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker