Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0035859 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
design defect | [Retail Modules] StoreServer | minor | have not tried | 2017-04-27 13:27 | 2017-05-11 14:22 | |||
Reporter | mtaal | View Status | public | |||||
Assigned To | mtaal | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | Fixed in SCM revision | e5ffcafbebca | ||||
Projection | none | ETA | none | Target Version | RR17Q2.1 | |||
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 | 0035859: Transition to online should only wait for store_data to replicate not other channels | |||||||
Description | The transition to online logic waits for all data to be synced before going online [1]. This is too pessimistic, this logic/rule only applies to business data which is located in the Store Data channel. [1] https://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/file/5cd4472a8d57/src/org/openbravo/retail/storeserver/synchronization/SyncStatusOnlineTransitionHandler.java#l65 [^] | |||||||
Steps To Reproduce | Multi-server Go offline with store server Change a mobile server definition in the store Go online, see that it waits for the change to replicate | |||||||
Proposed Solution | Only consider Store Data channel in the query here [2] [2] https://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/file/5cd4472a8d57/src/org/openbravo/retail/storeserver/synchronization/RetrieveSyncStatus.java#l63 [^] | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | |
Notes | |
(0096231) hgbot (developer) 2017-04-27 13:51 |
Repository: erp/pmods/org.openbravo.retail.storeserver.synchronization Changeset: e5ffcafbebca2b084ec6162718b8bbf35d9e1cb5 Author: Martin Taal <martin.taal <at> openbravo.com> Date: Thu Apr 27 13:29:48 2017 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/rev/e5ffcafbebca2b084ec6162718b8bbf35d9e1cb5 [^] Fixes issue 35859: Transition to online should only wait for store_data to replicate not other c Add channel_id to where clause in query, reformatting --- M src/org/openbravo/retail/storeserver/synchronization/RetrieveSyncStatus.java --- |
(0096529) AugustoMauch (manager) 2017-05-11 14:22 |
Code reviewed and verified |
Issue History | |||
Date Modified | Username | Field | Change |
2017-04-27 13:27 | mtaal | New Issue | |
2017-04-27 13:27 | mtaal | Assigned To | => mtaal |
2017-04-27 13:27 | mtaal | Triggers an Emergency Pack | => No |
2017-04-27 13:27 | mtaal | Target Version | RR17Q3 => RR17Q2.1 |
2017-04-27 13:51 | hgbot | Checkin | |
2017-04-27 13:51 | hgbot | Note Added: 0096231 | |
2017-04-27 13:51 | hgbot | Status | new => resolved |
2017-04-27 13:51 | hgbot | Resolution | open => fixed |
2017-04-27 13:51 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/rev/e5ffcafbebca2b084ec6162718b8bbf35d9e1cb5 [^] |
2017-05-01 16:39 | mtaal | Review Assigned To | => AugustoMauch |
2017-05-11 14:22 | AugustoMauch | Note Added: 0096529 | |
2017-05-11 14:22 | AugustoMauch | Status | resolved => closed |
Copyright © 2000 - 2009 MantisBT Group |