0034672: Default page size (35000) of initial data load is too high for high volume cases

Description

The default page size to load data from the backend is 35000 [1]. When there are many products or BPs this will load a big batch to the client to be inserted there. This insert takes a long time and requires lots of memory from the browser.

With 70000+ products load times were 15 mins or so.

The attached patch has been working correctly at a customer (hipertextil) for many months.

[1]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/7f3a3dde78ff/web/org.openbravo.mobile.core/source/data/ob-datasource.js#l495 [^]

Steps To Reproduce

Create a large set of products (70000) and put in assortment
clean cache
login to webpos
see that products get loaded in batches of 35000

Proposed Solution

See attached dif

It makes sense to backport this issue to previous releases Q3 and further.

Tags

No tags attached.

Attached Files

decreasePaginationSize_mobile_core.diff [^] (602 bytes) 2016-12-02 09:05 [Show Content] [Hide Content]

diff --git a/web/org.openbravo.mobile.core/source/data/ob-datasource.js b/web/org.openbravo.mobile.core/source/data/ob-datasource.js
--- a/web/org.openbravo.mobile.core/source/data/ob-datasource.js
+++ b/web/org.openbravo.mobile.core/source/data/ob-datasource.js
@@ -446,7 +446,7 @@
 
     this.cache = null;
 
-    handleIncrementalRequest(35000, 0, params, incremental);
+    handleIncrementalRequest(10000, 0, params, incremental);
   };
 
   OB.DS.DataSource.prototype.find = function (filter, callback) {
@@ -470,4 +470,4 @@
       }, this);
     }
   };
-}());
\ No newline at end of file
+}());

Relationships [ Relation Graph ] [ Dependency Graph ]

duplicate of

defect

0034669

RR17Q1

closed

migueldejuana

ProcessHQLQuery returns duplicates causing errors when inserting data and really slow login with many products

Notes
(0092977) marvintm (manager) 2016-12-23 14:26	This issue has been fixed as part of the fix for the issue 34669.

Issue History
Date Modified	Username	Field	Change
2016-12-02 09:05	mtaal	New Issue
2016-12-02 09:05	mtaal	Assigned To	=> Retail
2016-12-02 09:05	mtaal	File Added: decreasePaginationSize_mobile_core.diff
2016-12-02 09:05	mtaal	Triggers an Emergency Pack	=> No
2016-12-23 14:26	marvintm	Review Assigned To	=> marvintm
2016-12-23 14:26	marvintm	Relationship added	duplicate of 0034669
2016-12-23 14:26	marvintm	Note Added: 0092977
2016-12-23 14:26	marvintm	Status	new => closed
2016-12-23 14:26	marvintm	Resolution	open => duplicate