Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0034860 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
feature request | [Retail Modules] StoreServer | minor | have not tried | 2017-01-07 12:28 | 2017-01-11 17:24 | |||
Reporter | mtaal | View Status | public | |||||
Assigned To | mtaal | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | Fixed in SCM revision | |||||
Projection | none | ETA | none | Target Version | RR17Q2 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | AugustoMauch | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0034860: Improve fast/forced replication after a ticket has been send to the server | |||||||
Description | There is specific code in the SynchronizedServerProcessCaller that waits for the ticket to be synchronized from the central to the store server [1]. Some store server test cases log an error when waiting more than 6 seconds for data to be replicated from central to store [2]. This can be caused by not enough threads being available [3]. Or maybe we should wait longer than the push/route interval times. gr. Martin [1] https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/7097e311033f/src/org/openbravo/mobile/core/servercontroller/SynchronizedServerProcessCaller.java#l121 [^] [2] https://ci.openbravo.com/view/storeserver/job/ret-ms-store1-pgsql/498/testReport/org.openbravo.test.mobile.retail.multiserver.selenium.tests.store1/MS001_SalesFlow/test/ [^] [3] https://github.com/JumpMind/symmetric-ds/blob/3.8/symmetric-core/src/main/java/org/jumpmind/symmetric/service/impl/PushService.java#L123 [^] | |||||||
Steps To Reproduce | See the store server test environment | |||||||
Proposed Solution | Change the default number of available threads to 3 for the push service. One thread is for the standard periodical run, the other 2 threads are then for instant pushing. Change the wait procedure so that it first tries 20 times waiting 300 millis and then wait 20 times waiting 600 millis each. This will be waiting 18 seconds which is 8 seconds more than the push interval. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | |
Notes | |
(0093241) hgbot (developer) 2017-01-07 12:54 |
Repository: erp/pmods/org.openbravo.replication.symmetricds Changeset: 4557d1c17a882d22e614d0a777b7e5f5b002c49c Author: Martin Taal <martin.taal <at> openbravo.com> Date: Sat Jan 07 12:54:22 2017 +0100 URL: http://code.openbravo.com/erp/pmods/org.openbravo.replication.symmetricds/rev/4557d1c17a882d22e614d0a777b7e5f5b002c49c [^] Related to issue 34860: Improve fast/forced replication after a ticket has been send to the server Set default push threads to 3 to be more flexible in pushing data on request --- M src/org/openbravo/replication/symmetricds/templates/engine-properties.ftl --- |
(0093242) hgbot (developer) 2017-01-07 12:55 |
Repository: erp/pmods/org.openbravo.mobile.core Changeset: 137386ccc39da77c09f82bf2c91ccb14692f7ee8 Author: Martin Taal <martin.taal <at> openbravo.com> Date: Sat Jan 07 12:55:05 2017 +0100 URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/137386ccc39da77c09f82bf2c91ccb14692f7ee8 [^] Related to issue 34860: Improve fast/forced replication after a ticket has been send to the server Do the wait for data to be replicated in 2 steps, first waiting 300 ms per try, then 600 ms per try. --- M src/org/openbravo/mobile/core/servercontroller/SynchronizedServerProcessCaller.java --- |
(0093243) hgbot (developer) 2017-01-07 13:00 |
Repository: erp/pmods/org.openbravo.mobile.core Changeset: 85a9a96476f8338e56f2d04429d8188884d654de Author: Martin Taal <martin.taal <at> openbravo.com> Date: Sat Jan 07 13:00:33 2017 +0100 URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/85a9a96476f8338e56f2d04429d8188884d654de [^] Related to issue 34860: Improve fast/forced replication after a ticket has been send to the server Removed unintended System.err.println line --- M src/org/openbravo/mobile/core/servercontroller/SynchronizedServerProcessCaller.java --- |
(0093341) mtaal (manager) 2017-01-11 11:25 |
Resolved by previous commits |
(0093369) AugustoMauch (manager) 2017-01-11 17:24 |
Code reviewed |
Issue History | |||
Date Modified | Username | Field | Change |
2017-01-07 12:28 | mtaal | New Issue | |
2017-01-07 12:28 | mtaal | Assigned To | => mtaal |
2017-01-07 12:28 | mtaal | Triggers an Emergency Pack | => No |
2017-01-07 12:54 | hgbot | Checkin | |
2017-01-07 12:54 | hgbot | Note Added: 0093241 | |
2017-01-07 12:55 | hgbot | Checkin | |
2017-01-07 12:55 | hgbot | Note Added: 0093242 | |
2017-01-07 13:00 | hgbot | Checkin | |
2017-01-07 13:00 | hgbot | Note Added: 0093243 | |
2017-01-11 11:25 | mtaal | Status | new => scheduled |
2017-01-11 11:25 | mtaal | Note Added: 0093341 | |
2017-01-11 11:25 | mtaal | Status | scheduled => resolved |
2017-01-11 11:25 | mtaal | Resolution | open => fixed |
2017-01-11 11:25 | mtaal | Review Assigned To | => AugustoMauch |
2017-01-11 17:24 | AugustoMauch | Note Added: 0093369 | |
2017-01-11 17:24 | AugustoMauch | Status | resolved => closed |
Copyright © 2000 - 2009 MantisBT Group |