Skip to content
This repository has been archived by the owner on Dec 7, 2018. It is now read-only.

Implement a DCell registry #14

Open
aflatter opened this issue Jul 8, 2013 · 10 comments
Open

Implement a DCell registry #14

aflatter opened this issue Jul 8, 2013 · 10 comments
Assignees

Comments

@aflatter
Copy link
Member

aflatter commented Jul 8, 2013

All current DCell registries depend on a separate service. A floss-based registry would be able to run within each DCell node.

@ghost ghost assigned aflatter Jul 8, 2013
@aflatter
Copy link
Member Author

aflatter commented Jul 8, 2013

This is blocked by #7: Two nodes starting on separate processes will most definitely time out on start-up. #6 would be really useful too.

@aflatter
Copy link
Member Author

aflatter commented Jul 8, 2013

Work happening on the 14-dcell-registry branch.

@halorgium
Copy link
Contributor

@aflatter wahoo!

@tarcieri
Copy link
Member

tarcieri commented Jul 8, 2013

We might want to clean up the registry API before this happens

@aflatter
Copy link
Member Author

aflatter commented Jul 8, 2013

Any pointers to documentation of those plans?

@tarcieri
Copy link
Member

tarcieri commented Jul 8, 2013

Not really, but perhaps as an implementer of a registry you might have some opinions. The problem with the existing API is that it couples two similar concerns into a single class (node registry and global registry). Perhaps they should simply be the same thing.

@aflatter
Copy link
Member Author

aflatter commented Jul 8, 2013

Yes, noticed that. I may take a stab at refactoring this :-)

@halorgium
Copy link
Contributor

What @tarcieri is talking about is probably the change I made to @fujin's DCell commit.
celluloid/dcell@cd8bc0f

@tarcieri
Copy link
Member

tarcieri commented Jul 8, 2013

@halorgium yes, this! 😺

@fujin
Copy link

fujin commented Jul 9, 2013

🤘

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

4 participants