search

nextcloud / spreed

🗨️ Nextcloud Talk – chat, video & audio calls for Nextcloud
https://nextcloud.com/talk
GNU Affero General Public License v3.0
1.61k stars 429 forks source link

Chat should not poll the server but use events with the standalone signaling server #624

Open fancycode opened 6 years ago

fancycode commented 6 years ago

The chat currently polls for new messages. When using the standalone signaling server, this should be changed to be event based to reduce the server/database load and improve latency.

nickvergessen commented 6 years ago

Hmm the chat backend is not settled yet as far as I understood.

nickvergessen commented 5 years ago

From my POV we basically listen to the event for new chat/system messages, add a "pull for messages please" message to the signaling and remove the chat polling. The problem I see with this is that the ping was replaced with the chat polling, so this needs to be replaced/kept in mind too.

fancycode commented 5 years ago

The standalone signaling server "pings" connected clients internally, so this shouldn't be a problem.

PVince81 commented 3 years ago

Just noticed this happening and was wondering why we still poll.

I didn't see any signaling events for message sending, so I guess it's not implemented yet in the signaling server ?

nickvergessen commented 3 years ago

It's not implemented in the signaling backend in talks php. But also it's something for later

tcitworld commented 7 months ago

But also it's something for later

Is this still valid? What blocks this?

nickvergessen commented 7 months ago

Allocating time on all clients. It's still favored for the future, but nothing short term, but it was discussed recently again as potential improvement for scaling.