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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0040994
TypeCategorySeverityReproducibilityDate SubmittedLast Update
design defect[Retail Modules] StoreServerminorhave not tried2019-05-31 15:002019-06-08 16:54
ReportermtaalView Statuspublic 
Assigned Tomtaal 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0040994: Store server transition to offline has too long wait time, uses an old preference and can be simplified

DescriptionThe transition to offline process does different kinds of logic during the transition to offline which is not needed as it is already done before going to transition to offline:
- the intervaltime at the customer was set at 10 secs which was below the treshold in the code [6], so transition to offline always took 1 minute which is too long.
- checkoffline handlers [1] is already done here [2]
- it is using a wrong service to check cs availability [3], it should use [4]
- its checks stockmd to see if it is online or offline [5] but the decision to go offline is already made before so can't be repeated as one server can say yes go offline, while another says not can't go offline.

[1]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/ca7eed56ca97/src/org/openbravo/mobile/core/servercontroller/MobileMainServerCheck.java#l198 [^]

[2]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/ca7eed56ca97/src/org/openbravo/mobile/core/servercontroller/MobileServerController.java#l486 [^]

[3]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/ca7eed56ca97/src/org/openbravo/mobile/core/servercontroller/MobileMainServerCheck.java#l231 [^]

[4]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/tip/src/org/openbravo/mobile/core/servercontroller/MobileServerRequestExecutor.java#l310 [^]

[5]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/ca7eed56ca97/src/org/openbravo/mobile/core/servercontroller/MobileMainServerCheck.java#l206 [^]

[6]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/ca7eed56ca97/src/org/openbravo/mobile/core/servercontroller/MobileMainServerCheck.java#l72 [^]
Steps To ReproduceInstall multi-server
Proposed SolutionJust wait a preset configured time and then go offline. The wait time is needed to give the cs some time to sync latest informatiion
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to feature request 0040648 closedmtaal Make the store server more sensitive to overloaded cs 

-  Notes
(0112301)
mtaal (manager)
2019-05-31 15:01
edited on: 2019-05-31 15:01

 Changeset already reviewed
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/diff/790dfcea17be/src/org/openbravo/mobile/core/servercontroller/MobileMainServerCheck.java [^]
(0112514)
mtaal (manager)
2019-06-08 16:54

 changeset already reviewed

- Issue History
Date Modified Username Field Change
2019-05-31 15:00 mtaal New Issue
2019-05-31 15:00 mtaal Assigned To => mtaal
2019-05-31 15:00 mtaal Triggers an Emergency Pack => No
2019-05-31 15:01 mtaal Status new => scheduled
2019-05-31 15:01 mtaal Note Added: 0112301
2019-05-31 15:01 mtaal Note Edited: 0112301 View Revisions
2019-05-31 15:01 mtaal Status scheduled => resolved
2019-05-31 15:01 mtaal Resolution open => fixed
2019-05-31 15:01 mtaal Relationship added related to 0040648
2019-05-31 15:05 mtaal Description Updated View Revisions
2019-06-08 16:54 mtaal Note Added: 0112514
2019-06-08 16:54 mtaal Status resolved => closed

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker