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

Ensure nodes can reconnect after dropping out #13

Merged
merged 6 commits into from
Apr 15, 2016
Merged

Ensure nodes can reconnect after dropping out #13

merged 6 commits into from
Apr 15, 2016

Conversation

whitfin
Copy link
Owner

@whitfin whitfin commented Apr 14, 2016

This is based around #12 and corrects several issues in remote caches, namely;

  • fixes an issue where caches were not linked to supervisors correctly
  • fixes an issue where nodes are not brought into the cluster
  • fixes an issue where tables are not replicated across the cluster
  • adds a new add_node/2 function to add a node to a cache at runtime
  • adds a new start/1 function to start a cache without proc linking
  • adds new integration tests based around adding remote nodes

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 84e7051 on issue-12 into 403e1fa on master.

@whitfin
Copy link
Owner Author

whitfin commented Apr 14, 2016

I'm gonna let this soak for a little while and come back to it fresh before reviewing. The test setup is kinda flaky but seems to work 9 times out of 10 so is sufficient (at least it's good enough for me currently).

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling e5a8ce6 on issue-12 into 403e1fa on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 19b02c1 on issue-12 into 403e1fa on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 24d875d on issue-12 into 403e1fa on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling acfdea4 on issue-12 into 403e1fa on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 15b21fb on issue-12 into 403e1fa on master.

@whitfin whitfin merged commit 9443af4 into master Apr 15, 2016
@whitfin whitfin deleted the issue-12 branch April 15, 2016 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants