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

Cloud Sync #342

Merged
merged 3 commits into from
Feb 23, 2018
Merged

Cloud Sync #342

merged 3 commits into from
Feb 23, 2018

Conversation

jnunemaker
Copy link
Collaborator

This uses the sync adapter for cloud. It allows people to set a local adapter to keep in sync with remote. The default is in memory which is similar to using the active support in memory caching that we were recommending for cloud. The primary difference is that this is resilient to cloud being unavailable because exceptions are rescued and the local is persisted across restarts (assuming you are using active record, redis, mongo, or whatever instead of the default memory adapter). It also persists longer because the in memory cache expires, but this does not.

This improves latency and resiliency. Latency is improved because cloud
now uses a sync adapter with reads going to the local. Local reads mean
low latency.

Resiliency is improved because reads go to local which means if cloud is
unavailable most reads will be fine other than a few slow ones when
syncs are attempted (because they are not in background thread
currently).
@jnunemaker jnunemaker self-assigned this Feb 23, 2018
@jnunemaker jnunemaker merged commit cce96d4 into master Feb 23, 2018
@jnunemaker jnunemaker deleted the cloud-sync branch February 23, 2018 03:31
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

Successfully merging this pull request may close these issues.

1 participant