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

Merge meteor subscriptions into redux #35

Open
Chun-Yang opened this issue Aug 9, 2016 · 1 comment
Open

Merge meteor subscriptions into redux #35

Chun-Yang opened this issue Aug 9, 2016 · 1 comment

Comments

@Chun-Yang
Copy link
Contributor

  1. use autorun to emit actions and update collection data in store
  2. when we dispatch an action that require the data, we pass the data as part of the action
  3. for param-store, we always pass the previous state into the action
@Chun-Yang
Copy link
Contributor Author

Potential problem: performance penalty. We may subscribe to a lot of records. Since the state is not mutable, whenever we do an update, all of the data are re-written into memory.

TODO: need stress test.

Maybe we should keep meteor subscriptions out of the store.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant