932207 - Presence/Signaling server

Status: RESOLVED WONTFIX

(Hello (Loop) :: General, defect)

Description

[Tarek Ziadé (:tarek)]

The first load tests seem to indicate that adding a bunch of connected users significantly reduce the server speed.

I highly suspect the issue to be the broadcasting of the presence messages (userLeft, userJoined) because those messages are duplicated for each connected user in a for loop. 

I have no good knowledge of the app yet, but my first impression is that we could remove those loops and keep the broadcasted events in a single place, and decide what events to send to a user depending on the timestamp of their sign in.

The load test I am running is here: https://github.com/tarekziade/talkilla-loadtest/blob/master/test_talkilla.py#L67

using http://loads.readthedocs.org

Comment 1

[Tarek Ziadé (:tarek)]

Adding node-profiler in my instance to confirm/infirm this

Comment 2

[Mark Banner (:standard8)]

This is wontfix at the current time due to looking at a revised architecture.