Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

read only users don't need "really-real time" as much as contributing authors. Let's use that to our advantage. #18

Open
JohnMcLear opened this issue Oct 26, 2020 · 2 comments
Labels
enhancement New feature or request

Comments

@JohnMcLear
Copy link
Member

JohnMcLear commented Oct 26, 2020

Given our current SocketIO 10k msgs/sec constraint it may make sense to find a way to make non-contributing "lurkers" able to receive "batches" of changesets as opposed to "really-real-time"..

The reasoning is essentially if instead of a msg every 10ms to a lurker, you do it every 100ms as a batch of changesets you reduce socketio server load by 10x and the latency is/should be barely noticeable to someone lurking...

Something to consider anyway and might not fall within the remit of this plugin.

Also I'm really keen on moving away from SocketIO which would have the most meaningful impact on Etherpad performance.

@JohnMcLear JohnMcLear added the enhancement New feature or request label Oct 26, 2020
@rhansen
Copy link
Member

rhansen commented Oct 26, 2020

This will have to be fixed in Etherpad core. Other changes that would reduce overhead from read-only users:

  • Don't show read-only users in the user list.
  • Don't create an author ID for read-only users.

@JohnMcLear
Copy link
Member Author

Agreed, note that "Don't show read-only users in the user list.", it might be nice to see "X number of watchers"

@rhansen rhansen transferred this issue from ether/ep_readonly_guest May 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants