Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Add option to configure global syncFrequency #39

Open
wtrocki opened this issue Oct 9, 2017 · 2 comments
Open

Add option to configure global syncFrequency #39

wtrocki opened this issue Oct 9, 2017 · 2 comments

Comments

@wtrocki
Copy link
Member

wtrocki commented Oct 9, 2017

Current sync api requires us to register dataset on the server to change sync frequency.

  sync.api.init('myDataset', {
    syncFrequency: 10 // seconds
  }, function() {});

Problem is that this will require users to know their datasets all the time and init them directly in the server. I think it will be nice to have global sync frequency setup on the server so users will still have configuration to tweak for all the requests. Alternative will be to support passing syncFrequency in the client.

By default now everything will be done even 10 seconds which may impact performance.

@wtrocki
Copy link
Member Author

wtrocki commented Oct 9, 2017

ping @david-martin @aidenkeating

@paolobueno
Copy link
Contributor

👍 I think having a global overridable default for new datasets would be helpful.

I think a separate frequency per client might be a much bigger change, but I think there are scenarios where this level of fine-grain could help with valuable tweaks.

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

No branches or pull requests

2 participants