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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0037249
TypeCategorySeverityReproducibilityDate SubmittedLast Update
feature request[Retail Modules] StoreServerminorhave not tried2017-11-07 13:372017-11-08 11:44
ReportermtaalView Statuspublic 
Assigned Tomtaal 
PrioritynormalResolutionfixedFixed in VersionRR18Q1
StatusclosedFix in branchFixed in SCM revisionf21e14b092a0
ProjectionnoneETAnoneTarget Versionpi
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned ToAugustoMauch
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0037249: For server to server requests set read and connection time out

DescriptionThe case is described in this [1] document. When doing server to server requests [2] no time out is set. This means that the system uses the default (infinite wait) for 2 timeouts:
- connect timeout: timeout for creating the request connection between servers
- read timeout: timeout in which the calling server waits for the receiving server to process the request. The read timeout applies after the connection to the other server has been made.

The system default is infinite wait.

[1]
https://docs.google.com/document/d/1ZNgmx_AWu0t8zBuXwgkeHH6sFsb_X-G8k7KyWxkAaHQ/edit?ts=5a018088# [^]
Steps To ReproduceInstall multi-server
Put a breakpoint in the request executor when it catches an exception [1]
Put a breakpoint in the central server OrderLoader
Goto webpos, enter ticket and submit
skip the breakpoint in the store server, let it wait on the breakpoint in the CS.

The request from SS to CS does not timeout. If it would timeout a timeout exception should be caught at [1]

[2]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/ca39e197ccae/src/org/openbravo/mobile/core/servercontroller/MobileServerRequestExecutor.java#l197 [^]
Proposed SolutionDefault connect timeout at 5 seconds and the readtimeout at 15 minutes. Do the change here [1]. Make the timeouts configurable if possible.

[1]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/ca39e197ccae/src/org/openbravo/mobile/core/servercontroller/MobileServerRequestExecutor.java#l70 [^]
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to defect 0036884 closedmtaal [SERQA 3289] If we stop the instance instead of tomcat we get a timeout paying a ticket 
blocks feature request 0037379 closedmtaal In mobile server request executor when read time out occurs then do not go offline 

-  Notes
(0100307)
mtaal (manager)
2017-11-07 15:07

 Docs added here:
http://wiki.openbravo.com/wiki/Store_and_Multi_Server_Preferences#Behavioral_Preferences [^]
(0100328)
hgbot (developer)
2017-11-08 09:01

 Repository: erp/pmods/org.openbravo.mobile.core
Changeset: f21e14b092a045a017abaf9b5d96ed76e14c7dd0
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Wed Nov 08 09:00:45 2017 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/f21e14b092a045a017abaf9b5d96ed76e14c7dd0 [^]

fixes issue 37249, fixes issue 37227: For server to server requests set read and connection time out
Set defaults for read and connection timeout and make them configurable
Make sure max retries has minimum value of 1

---
M src-db/database/sourcedata/AD_REF_LIST.xml
M src/org/openbravo/mobile/core/servercontroller/MobileServerRequestExecutor.java
---
(0100338)
AugustoMauch (administrator)
2017-11-08 11:44

 Code reviewed and verified

- Issue History
Date Modified Username Field Change
2017-11-07 13:37 mtaal New Issue
2017-11-07 13:37 mtaal Assigned To => mtaal
2017-11-07 13:37 mtaal Triggers an Emergency Pack => No
2017-11-07 15:07 mtaal Note Added: 0100307
2017-11-07 15:07 mtaal Status new => acknowledged
2017-11-07 15:08 mtaal Review Assigned To => AugustoMauch
2017-11-08 09:01 hgbot Checkin
2017-11-08 09:01 hgbot Note Added: 0100328
2017-11-08 09:01 hgbot Status acknowledged => resolved
2017-11-08 09:01 hgbot Resolution open => fixed
2017-11-08 09:01 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/f21e14b092a045a017abaf9b5d96ed76e14c7dd0 [^]
2017-11-08 11:44 AugustoMauch Note Added: 0100338
2017-11-08 11:44 AugustoMauch Status resolved => closed
2017-11-08 11:44 AugustoMauch Fixed in Version => RR18Q1
2017-11-19 23:55 mtaal Relationship added has duplicate 0036884
2017-11-19 23:56 mtaal Relationship deleted has duplicate 0036884
2017-11-19 23:56 mtaal Relationship added related to 0036884
2017-11-24 07:11 mtaal Relationship added blocks 0037379

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker