Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0037305Retail ModulesWeb POSpublic2017-11-16 07:392017-12-22 08:42
Reporterguillermogil 
Assigned Tomarvintm 
PriorityimmediateSeveritymajorReproducibilitysometimes
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR17Q3.2Fixed in VersionRR17Q3.3 
Merge Request Status
Review Assigned Tomigueldejuana
OBNetwork customer
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2016-01-05
Regression introduced in releaseRR16Q2
Regression introduced by commithttps://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/727a4f04e3e023624bbdc36cdb682e95304129ad [^]
Triggers an Emergency PackNo
Summary0037305: Java Heap Memory can rise on login if batch size is big
DescriptionJava Heap Memory can rise on login if batch size is big and you have a big batch size and an small xmx.
Steps To ReproduceWith a lot of products, big batch size (> 60000) and an small xmx in CATALINA_OPTS (1GB)
Try to login in WebPOS
You will have a Java heap memory.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0037301 closed marvintm Java Heap Memory can rise on login if batch size is big 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-11-16 10:27marvintmTypedesign defect => backport
2017-11-16 10:27marvintmTarget Version => RR17Q3.2
2017-12-19 18:40marvintmAssigned ToRetail => marvintm
2017-12-19 18:50hgbotCheckin
2017-12-19 18:50hgbotNote Added: 0101149
2017-12-19 18:50hgbotStatusscheduled => resolved
2017-12-19 18:50hgbotResolutionopen => fixed
2017-12-19 18:50hgbotFixed in SCM revision => http://code.openbravo.com/retail/backports/3.0RR17Q3.2/org.openbravo.mobile.core/rev/3840f870f2cb804eacae68a42667e1d72c8e8c28 [^]
2017-12-22 08:42migueldejuanaReview Assigned To => migueldejuana
2017-12-22 08:42migueldejuanaNote Added: 0101196
2017-12-22 08:42migueldejuanaStatusresolved => closed
2017-12-22 08:42migueldejuanaFixed in Version => RR17Q3.3

Notes
(0101149)
hgbot   
2017-12-19 18:50   
Repository: retail/backports/3.0RR17Q3.2/org.openbravo.mobile.core
Changeset: 3840f870f2cb804eacae68a42667e1d72c8e8c28
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Tue Dec 19 18:50:14 2017 +0100
URL: http://code.openbravo.com/retail/backports/3.0RR17Q3.2/org.openbravo.mobile.core/rev/3840f870f2cb804eacae68a42667e1d72c8e8c28 [^]

Fixed issue 37305. Avoid usage of intermediate StringWriters so that response doesn't take too much heap.
> The main problem is that the two intermediate StringWriters accumulate the whole response before it is sent back to the client. This causes Java Heap memory problems in case the batching is set to a big amount, and concurrency is high.
> The solution is to remove these intermediate StringWriters. We have verified that the messages are not lost even in the case of low level database problems, because the client correctly detects the problem, and doesn't remove the records, and instead tries to send them again in the next synchronization.

---
M src/org/openbravo/mobile/core/process/MobileService.java
M src/org/openbravo/mobile/core/process/MobileServiceProcessor.java
---
(0101196)
migueldejuana   
2017-12-22 08:42   
Reviewed