Andrzej Wójcik (Tigase) opened 3 years ago
|
|
The logic of
By setting This new code is available in a feature branch and PR was created at https://github.com/tigase/tigase-server/pull/150 @wojtek Please review this logic and code before we merge this to master. |
|
I reviewed the code and it looks ok. Added a couple minute comments. Please also include this in the documentation (could be based on your comment above) |
Type |
Bug
|
Priority |
Normal
|
Assignee | |
Version |
tigase-server-8.3.0
|
Spent time |
1h 45m
|
Issue Votes (0)
Watchers (0)
When no. of unacked stanzas in the Stream Management queue is bigger than configured
ack_request_count
then the server sends<r/>
packet for each new packet sent to the client. This is most likely overkill, increases data traffic sent to the client, and duplicates no. of stanza sent to the client (ie. mobile device with poor connectivity).Moreover, this forces client to send
<a/>
response stanza for each received<r/>
stanza, causing increased traffic sent to the XMPP server.All of that with long round trip time between client and the server, causes saturation of small network buffer used by the server for clients connections and increases delay, ie. during MAM sync.