-
Notifications
You must be signed in to change notification settings - Fork 7
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
Resumable DDP subscriptions #52
Comments
It there a message from @glasser missing in the description above? |
just added references, and removed the label "MDG dont agree" since its not 100% clear |
This could also be used for server side rendering. For instance the server send the 10 first items of a list in the initial HTTP response. Then the client could resume this DDP connection so we don't have to send the same data twice (is that still the case in |
We don't have any kind of resuming in fast-render but it does do conflict resolving upto some point. If we need to do resumable stuff I think we need to manage changes in a way how git works. So, that's a big topic. |
What has |
Its not possible to resume a DDP connection - I'm mainly thinking about pub/sub.
Why?
Brainstorm:
The server could have a different way to keep track of subscriptions - perhaps the client is better fittet requesting the data not on the client:
The client could tell the server what subscriptions are on the client - this way the server could query the db directly using the
publish query
on the client, theserver time stamp
and theresumed publish query
.I'll prop. write a prototype of this since its part of getting ground:db to work better with subscriptions and the iron:router.
This is not ideal to solve in packages since it requires replacing things in ddp/livedata etc. but thats been the message from @glasser so far:
Note: The initial code indicating resumable subscriptions were swiped from Meteor approx: a year ago (or something - I havent tracked down a reference, but its part of a clean up by glasser)
Note about resume:
Ref server resume:
Ref about server timestamp:
The text was updated successfully, but these errors were encountered: