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.

Sign up for GitHub

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

Jump to bottom

[Perf] Buffer client>server events #16

Open
ptaoussanis opened this issue Mar 30, 2014 · 0 comments
Open

[Perf] Buffer client>server events #16

ptaoussanis opened this issue Mar 30, 2014 · 0 comments
Labels
pull requests welcome

Comments

@ptaoussanis
Copy link
Member

This is far less important than buffering for server>user events (which we now have), but it'd still be a big efficiency win - esp. for Ajax + for bandwidth minimization on mobiles.

It's even possible to offer this for events with callbacks if we're prepared to allow actual timeouts to vary by [0,<max-buffer-window-size>] ms.

UPDATE: Have started some work for this on the dev branch. Shouldn't be hard to implement, just short on time atm.
@ptaoussanis ptaoussanis self-assigned this Mar 30, 2014
@ptaoussanis ptaoussanis removed this from the v1.1.0 - optimizations release milestone Aug 29, 2014
@ptaoussanis ptaoussanis changed the title Optimization: buffer client>server events w/o callbacks Optimization: buffer client>server events Sep 21, 2014
ptaoussanis added a commit that referenced this issue Sep 21, 2014
@ptaoussanis
#16: Get started on client-side buffering
b489fb0
ptaoussanis added a commit that referenced this issue Sep 22, 2014
@ptaoussanis
Get started on #16 (client-side buffering)
71e1216
tobias pushed a commit to tobias/sente that referenced this issue Jan 29, 2015
@ptaoussanis
Get started on taoensso#16 (client-side buffering)
2f691a2
@ptaoussanis ptaoussanis changed the title Optimization: buffer client>server events [Perf] Buffer client>server events Dec 10, 2015
@ptaoussanis ptaoussanis modified the milestone: v1.10.0 Jul 14, 2016
@ptaoussanis ptaoussanis removed their assignment Aug 5, 2016
@ptaoussanis ptaoussanis removed this from the v1.11.0 milestone Aug 31, 2016
ptaoussanis added a commit that referenced this issue Oct 26, 2016
@ptaoussanis
NB Experimental: new bulk pstr format to replace ppstr concept
3e9d87f 
Objectives:
  - [#276] Support easy+efficient generic un/pack caching.
    - In particular: support cache-friendly server->client event buffering.
  - [#16] Support client->server buffering with arb cbs.
  - Read+write performance.
  - Bandwidth efficiency.
  - Flexibility for possible future extensions/changes.
  - Human-readable on the wire (handy for debugging, etc.).
  - Lay groundwork for possible broadcast API fn.

Downsides:
  - Would involve a breaking incompatibility with old clients.
ptaoussanis added a commit that referenced this issue Oct 26, 2016
@ptaoussanis
NB Experimental: new bulk pstr format to replace ppstr concept
be9f009 
Objectives:
  - [#276] Support easy+efficient generic un/pack caching.
    - In particular: support cache-friendly server->client event buffering.
  - [#16] Support client->server buffering with arb cbs.
  - Read+write performance.
  - Bandwidth efficiency.
  - Flexibility for possible future extensions/changes.
  - Human-readable on the wire (handy for debugging, etc.).
  - Lay groundwork for possible broadcast API fn.

Downsides:
  - Would involve a breaking incompatibility with old clients.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
pull requests welcome
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ptaoussanis